On 20 November 2014 01:33, Kevin Hilman <khil...@kernel.org> wrote:
> Ulf Hansson <ulf.hans...@linaro.org> 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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to