RE: [PATCH v2 1/2] PM / Domains: Add OF support

[Kukjin Kim <kgene.kim@samsung.com>]

Rafael J. Wysocki wrote:
> 
> On Saturday, January 07, 2012, Thomas Abraham wrote:
> > A device node pointer is added to generic pm domain structure to
> associate
> > the domain with a node in the device tree. The platform code parses the
> > device tree to find available nodes representing the generic power
> domain,
> > instantiates the available domains and initializes them by calling
> > pm_genpd_init().
> >
> > Nodes representing the devices include a phandle of the power domain to
> > which it belongs. As these devices get instantiated, the driver code
> > checkes for availability of a power domain phandle, converts the phandle
> > to a device node and uses the new pm_genpd_of_add_device() api to
> > associate the device with a power domain.
> >
> > pm_genpd_of_add_device() runs through its list of registered power
> domains
> > and matches the OF node of the domain with the one specified as the
> > parameter. If a match is found, the device is associated with the
> matched
> > domain.
> >
> > Cc: Rafael J. Wysocki <rjw@sisk.pl>
> > Cc: Rob Herring <rob.herring@calxeda.com>
> > Cc: Grant Likely <grant.likely@secretlab.ca>
> > Signed-off-by: Thomas Abraham <thomas.abraham@linaro.org>
> 
> I can take this patch for 3.4, but your [2/2] depends on it, so I'm not
> sure how to handle that.  If you want me to take the other patch too,
> it'll need ACKs from the Exynos maintaniers.
> 
Hi Rafael,

Basically, looks ok to me.

But it would be good if [v3 2/2] patch of this series (this patch) could be
applied in samsung tree to avoid conflicts as it modifies many files. I
don't want to make a trouble when sending my tree to upstream. Of course, as
you said, this have a dependency with [v2 1/2]. So if you're ok, let me
create topic branch will have both(1/2 and 2/2) to merge in your tree.

How do you think?

Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.