On 20 November 2014 01:33, Kevin Hilman <khilman@xxxxxxxxxx> wrote: > Ulf Hansson <ulf.hansson@xxxxxxxxxx> writes: > >> 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. >> >> 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 >> that left to future patchsets to address. > > I think we need to get rid of the SoC specific code already. For > example, we're already seeing SoCs where the arm32 core is being > replaced by an arm64 core but the other IPs, and power-domain logic is > staying more or less the same. > >> It's not every users of genpd that are keen on using PM clocks thus we need >> to provide this a configuration option for genpd. > > Or more likely, probably some compatible string, or property in the > domain node. Grygorii, Arnd and myself were discussing this elsewhere > in the context of the TI Keystone2 PM domain support[1]. Thanks for pointing that out. It was actually the reason to why I posted this patchset now, I have been keeping the patches locally in my tree for too long. :-) Let me comment of that thread, instead of here. Kind regards Uffe -- 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