On Tue, Apr 21, 2015 at 1:25 AM, santosh shilimkar <santosh.shilimkar@xxxxxxxxxx> wrote: > On 4/20/2015 4:21 PM, Kevin Hilman wrote: >> >> Rajendra Nayak <rnayak@xxxxxxxxxxxxxx> writes: >> >>> Most users of PM clocks do the exact same thing in runtime callbacks. >> >> >> Probably because they were all copied from mach-davinci. ;) >> > Yep. ;-) If you're interested in the history, I did some digging last year: http://lists.infradead.org/pipermail/linux-arm-kernel/2014-March/242352.html >>> Provide default callbacks and cleanup the existing users >>> (keystone/davinci/omap1/sh) >> >> >> Very nice cleanup, Thanks! Note that the new code always has a dev_pm_domain, while the old code had it conditionally on CONFIG_PM. I don't think that matters much, as we seem to be having more and more systems that rely on CONFIG_PM=y... >> For the series: >> >> Reviewed-by: Kevin Hilman <khilman@xxxxxxxxxx> >> > Acked-by: Santosh Shilimkar <ssantosh@xxxxxxxxxx> Looks good to me, and works fine on (pre-Clock Domain) r8a7791/koelsch and r8a7740/armadillo-legacy (with PM Domains, but still relying on the drivers/sh/pm_runtime.c hack for devices in the C5 "always on" domain). This code is no longer used in multiplatform kernels on shmobile boards with real PM Domains. Acked-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html