Hi Ulf, On Wed, Nov 19, 2014 at 3:00 PM, Ulf Hansson <ulf.hansson@xxxxxxxxxx> wrote: > It's quite common for PM domains to use PM clocks. Typically from SOC specific > code, the per device PM clock list is created and pm_clk_suspend|resume() are > invoked to handle clock gating/ungating. > > A step towards consolidation is to integrate PM clock support into genpd, which > is what this patchset does. Thanks, this sounds like a good idea. > In this initial step, the calls to the pm_clk_suspend|resume() are handled > within genpd, but the per device PM clock list still needs to be created from > SOC specific code. It seems reasonable to have gendp to handle that as well, but s/gendp/genpd/ > that left to future patchsets to address. If/when genpd will handle the clock creation, perhaps the flag can just be replaced by a pointer to a list of clocks? Or do you have other uses in mind for adding the flags field now? 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-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html