From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kukjin Kim Subject: RE: [PATCH v2 1/2] PM / Domains: Add OF support Date: Tue, 17 Jan 2012 16:22:47 +0900 Message-ID: <001e01ccd4e8$d04b03a0$70e10ae0$%kim@samsung.com> References: <1325931055-14486-1-git-send-email-thomas.abraham@linaro.org> <1325931055-14486-2-git-send-email-thomas.abraham@linaro.org> <201201082223.18603.rjw@sisk.pl> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Return-path: In-reply-to: <201201082223.18603.rjw@sisk.pl> Content-language: ko Sender: linux-samsung-soc-owner@vger.kernel.org To: "'Rafael J. Wysocki'" , 'Thomas Abraham' Cc: linux-samsung-soc@vger.kernel.org, linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree-discuss@lists.ozlabs.org, rob.herring@calxeda.com, grant.likely@secretlab.ca, broonie@opensource.wolfsonmicro.com, kyungmin.park@samsung.com, s.nawrocki@samsung.com, patches@linaro.org List-Id: devicetree@vger.kernel.org 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 > > Cc: Rob Herring > > Cc: Grant Likely > > Signed-off-by: Thomas Abraham > > 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 , Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd.