[PATCH 1/2] regulator: max77686: Implement suspend disable for some LDOs
Javier Martinez Canillas
javier.martinez at collabora.co.uk
Tue Oct 21 02:54:16 PDT 2014
Hello Krzysztof,
On 10/21/2014 11:36 AM, Krzysztof Kozlowski wrote:
> On wto, 2014-10-21 at 11:10 +0200, Javier Martinez Canillas wrote:
>>
>> In general, could you please take a look to the latest patches I posted for the
>> max77802? They are in a topic branch in Mark's regulator tree [1]. I would like
>> both 77686 and 77802 drivers to handle things similarly even though I know that
>> the PMICs don't behave identically and that's why we decided to have different
>> drivers (although maybe that was a mistake and makes more sense to merge them).
>
> I looked at them. In this patchset I didn't want to implement fully the
> whole suspend-mode-thing. Just the first, easier part - set suspend
> disable.
>
Agreed, incremental support makes sense to me as well.
I also want to configure the regulator modes in suspend for the Snow Chromebook
using the same modes that are in the downstream Chrome OS tree and I think your
changes will be enough for that too.
> In general I would like to do it similar to your solution but that is
> work for future. Especially as I would like to wait for merging your
> "Add max77802 regulator operating mode support".
>
Perfect, thanks a lot for working on this!
> Best regards,
> Krzysztof
>
Best regards,
Javier
More information about the linux-arm-kernel
mailing list