From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ulf Hansson Subject: Re: [PATCH v3 2/9] PM / Domains: Add generic OF-based PM domain look-up Date: Tue, 9 Sep 2014 09:04:57 +0200 Message-ID: References: <1409838756-31963-1-git-send-email-ulf.hansson@linaro.org> <3075771.rIWPm62y5x@vostro.rjw.lan> <5191943.peyCs6gZVs@vostro.rjw.lan> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: In-Reply-To: <5191943.peyCs6gZVs@vostro.rjw.lan> Sender: linux-pm-owner@vger.kernel.org To: "Rafael J. Wysocki" Cc: Rob Herring , Len Brown , Pavel Machek , Greg Kroah-Hartman , "linux-pm@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , ACPI Devel Maling List , Geert Uytterhoeven , Kevin Hilman , Alan Stern , Daniel Lezcano , "devicetree@vger.kernel.org" , Linus Walleij , Simon Horman , Magnus Damm , Ben Dooks , Kukjin Kim , Stephen Boyd , Philipp Zabel , Mark Brown , Wolfram Sang List-Id: linux-acpi@vger.kernel.org On 8 September 2014 23:04, Rafael J. Wysocki wrote: > On Monday, September 08, 2014 09:26:20 AM Ulf Hansson wrote: >> On 8 September 2014 00:13, Rafael J. Wysocki wrote: >> > On Thursday, September 04, 2014 03:52:29 PM Ulf Hansson wrote: >> >> From: Tomasz Figa >> >> >> >> This patch introduces generic code to perform PM domain look-up using >> >> device tree and automatically bind devices to their PM domains. >> >> >> >> Generic device tree bindings are introduced to specify PM domains of >> >> devices in their device tree nodes. >> >> >> >> Backwards compatibility with legacy Samsung-specific PM domain bindings >> >> is provided, but for now the new code is not compiled when >> >> CONFIG_ARCH_EXYNOS is selected to avoid collision with legacy code. >> >> This will change as soon as the Exynos PM domain code gets converted to >> >> use the generic framework in further patch. >> >> >> >> Signed-off-by: Tomasz Figa >> >> Signed-off-by: Ulf Hansson >> >> [Ulf:Added attach|detach functions, fixed review comments] >> >> Tested-by: Philipp Zabel >> > >> > However, for this one (and consequently the rest) I need an ACK from the >> > people who maintain the bindings. >> >> These DT bindings have been discussed between Tomasz and devicetree >> maintainers previously. So I think there are fine. >> >> Also, when browsing the mail-archives, I found an ack from Rob Herrring: >> http://lists.infradead.org/pipermail/linux-arm-kernel/2014-March/237989.html >> >> I realize that I didn't put the DT maintainers on the to-line when >> posting this patch, but just the devicetree list. I have now included >> them on this reply, but maybe I should repost instead. What do you >> think? Are you happy with the above ack from Rob? > > That should be sufficient, but I wonder why you didn't add it to the patch > to start with? I didn't remember it being formally acked, until I decided to browsed the mail archives in detail. Sorry about that. Kind regards Uffe From mboxrd@z Thu Jan 1 00:00:00 1970 From: ulf.hansson@linaro.org (Ulf Hansson) Date: Tue, 9 Sep 2014 09:04:57 +0200 Subject: [PATCH v3 2/9] PM / Domains: Add generic OF-based PM domain look-up In-Reply-To: <5191943.peyCs6gZVs@vostro.rjw.lan> References: <1409838756-31963-1-git-send-email-ulf.hansson@linaro.org> <3075771.rIWPm62y5x@vostro.rjw.lan> <5191943.peyCs6gZVs@vostro.rjw.lan> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 8 September 2014 23:04, Rafael J. Wysocki wrote: > On Monday, September 08, 2014 09:26:20 AM Ulf Hansson wrote: >> On 8 September 2014 00:13, Rafael J. Wysocki wrote: >> > On Thursday, September 04, 2014 03:52:29 PM Ulf Hansson wrote: >> >> From: Tomasz Figa >> >> >> >> This patch introduces generic code to perform PM domain look-up using >> >> device tree and automatically bind devices to their PM domains. >> >> >> >> Generic device tree bindings are introduced to specify PM domains of >> >> devices in their device tree nodes. >> >> >> >> Backwards compatibility with legacy Samsung-specific PM domain bindings >> >> is provided, but for now the new code is not compiled when >> >> CONFIG_ARCH_EXYNOS is selected to avoid collision with legacy code. >> >> This will change as soon as the Exynos PM domain code gets converted to >> >> use the generic framework in further patch. >> >> >> >> Signed-off-by: Tomasz Figa >> >> Signed-off-by: Ulf Hansson >> >> [Ulf:Added attach|detach functions, fixed review comments] >> >> Tested-by: Philipp Zabel >> > >> > However, for this one (and consequently the rest) I need an ACK from the >> > people who maintain the bindings. >> >> These DT bindings have been discussed between Tomasz and devicetree >> maintainers previously. So I think there are fine. >> >> Also, when browsing the mail-archives, I found an ack from Rob Herrring: >> http://lists.infradead.org/pipermail/linux-arm-kernel/2014-March/237989.html >> >> I realize that I didn't put the DT maintainers on the to-line when >> posting this patch, but just the devicetree list. I have now included >> them on this reply, but maybe I should repost instead. What do you >> think? Are you happy with the above ack from Rob? > > That should be sufficient, but I wonder why you didn't add it to the patch > to start with? I didn't remember it being formally acked, until I decided to browsed the mail archives in detail. Sorry about that. Kind regards Uffe