Re: [PATCH V2] PM / Domains: Add pm_genpd_lookup() API to lookup domain by firmware node

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

 



On 1 December 2014 at 12:57, amit daniel kachhap
<amit.daniel@xxxxxxxxxxx> wrote:
> On Mon, Dec 1, 2014 at 3:56 PM, Ulf Hansson <ulf.hansson@xxxxxxxxxx> wrote:
>> On 29 November 2014 at 01:30, Rafael J. Wysocki <rjw@xxxxxxxxxxxxx> wrote:
>>> On Friday, November 28, 2014 11:38:35 AM Ulf Hansson wrote:
>>>> In a step to move away from using genpd's name based APIs, such as the
>>>> pm_genpd_add_subdomain_names(), provide an API to lookup an already
>>>> initialized generic PM domain by its firmware node.
>>>>
>>>> This API would typically be a called from SOC specific code, to fetch a
>>>> handle to the domain. Especially convenient to configure subdomains and
>>>> when the hierarchy of the domains are described in DT.
>>>>
>>>> Do note, before pm_genpd_init() is invoked to initialize a generic PM
>>>> domain, it's the callers responsibility to assign the new ->fwnode
>>>> pointer in the struct generic_pm_domain, to enable pm_genpd_lookup() to
>>>> find the domain.
>>>>
>>>> Signed-off-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx>
>>>
>>> I have no problems with that, but do you have a user for it?
>>
>> Amit Daniel Kachhap, posted a patchset which is adding subdomains
>> configured from DT for Exynos. It's from there the idea to this patch
>> comes from. Amit is about to post a new version, based on this patch.
>
> Hi Ulf,
>
> I was trying to use this API but i felt there is a better way of just exporting
> of_genpd_get_from_provider as a similar API is used to register the
> power domain.
>
> Check if this make sense www.spinics.net/lists/arm-kernel/msg382665.html.
>

That makes sense!

So, let's drop $subject patch.

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