On Thu, May 17, 2012 at 12:34 PM, Shilimkar, Santosh <[email protected]> wrote: > On Thu, May 17, 2012 at 4:12 AM, Kevin Hilman <[email protected]> wrote: >> Tero Kristo <[email protected]> writes: >> >>> From: Rajendra Nayak <[email protected]> >>> >>> 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. >>> >>> [[email protected]: minor cleanups] >>> Signed-off-by: Nishanth Menon <[email protected]> >>> Signed-off-by: Rajendra Nayak <[email protected]> >>> Signed-off-by: Santosh Shilimkar <[email protected]> >>> Signed-off-by: Tero Kristo <[email protected]> >> >> 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 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
