Re: [PATCH 0/3] PM / Domains: Add initial PM clock support to genpd

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux