> 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. Right, but since USE_PM_CLK_RUNTIME_OPS is actually a nop with !CONFIG_PM, we don't really need the checks anymore. > > 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> Thanks, Rajendra > > 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 > -- QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation -- 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