Re: [PATCHv2 04/19] ARM: OMAP4: PM: save/restore all DPLL settings in OFF mode

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

On Thu, May 17, 2012 at 12:34 PM, Shilimkar, Santosh
<santosh.shilimkar@xxxxxx> wrote:
> On Thu, May 17, 2012 at 4:12 AM, Kevin Hilman <khilman@xxxxxx> wrote:
>> Tero Kristo <t-kristo@xxxxxx> writes:
>>
>>> From: Rajendra Nayak <rnayak@xxxxxx>
>>>
>>> SAR/ROM code restores only CORE DPLL to its original state
>>> post wakeup from OFF mode.
>>> The rest of the DPLL's in OMAP4 platform (MPU/IVA/ABE/USB/PER)
>>> are saved and restored here during an OFF transition.
>>>
>>> [nm@xxxxxx: minor cleanups]
>>> Signed-off-by: Nishanth Menon <nm@xxxxxx>
>>> Signed-off-by: Rajendra Nayak <rnayak@xxxxxx>
>>> Signed-off-by: Santosh Shilimkar <santosh.shilimkar@xxxxxx>
>>> Signed-off-by: Tero Kristo <t-kristo@xxxxxx>
>>
>> Some general comments:
>>
>> - the register dump/print doesn't belong here.  If needed, should be a
>>  debug feature if needed.
>>
>> - Rather than hooking into omap4_enter_lowpower(), should use
>>  the cluster PM enter/exit notifier chain.
>>
> This is again specific to device OFF only and not related to CPU
> cluster state as such. So I don't think notifiers should be used here.
>
> O.w even when we attempt just MPU OSWR C-state, all these functions will
> get called in notifier chain.
>
Just a thought, we can have a separate notifier chain for device OFF. It can
allow use to get rid of 'enable_off_mode" kind of flags and can be
used by many drivers too.

Regards
Santosh

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel



[Linux ARM (vger)]     [Linux ARM MSM]     [Linux Omap]     [Linux Arm]     [Linux Tegra]     [Fedora ARM]     [eCos]     [Linux Fastboot]     [Gcc Help]     [Git]     [DCCP]     [IETF Announce]     [Security]     [PDAs]     [Linux]     [Linux MIPS]     [Yosemite Campsites]     [Photos]

Add to Google Follow linuxarm on Twitter