From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ulf Hansson Subject: Re: [PATCH v8 03/26] PM / Domains: Add generic data pointer to genpd_power_state struct Date: Mon, 25 Jun 2018 10:34:38 +0200 Message-ID: References: <20180620172226.15012-1-ulf.hansson@linaro.org> <20180620172226.15012-4-ulf.hansson@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: "Rafael J. Wysocki" Cc: "Rafael J . Wysocki" , Sudeep Holla , Lorenzo Pieralisi , Mark Rutland , Linux PM , Kevin Hilman , Lina Iyer , Lina Iyer , Rob Herring , Daniel Lezcano , Thomas Gleixner , Vincent Guittot , Stephen Boyd , Juri Lelli , Geert Uytterhoeven , Linux ARM , linux-arm-msm , Linux Kernel Mailing List List-Id: linux-arm-msm@vger.kernel.org On 24 June 2018 at 23:09, Rafael J. Wysocki wrote: > On Wed, Jun 20, 2018 at 7:22 PM, Ulf Hansson wrote: >> From: Lina Iyer >> >> Let's add a data pointer to the genpd_power_state struct, to allow >> platforms to store per state specific data. > > Can you please fold it into a patch actually using this pointer? Yep, no problem. Anyway, the change that uses the pointer is "[PATCH v8 21/26] drivers: firmware: psci: Add support for PM domains using genpd". Let's see if there is further comments, if not - perhaps you can squash $subject patch into that change? > >> Cc: Lina Iyer >> Signed-off-by: Lina Iyer >> Co-developed-by: Ulf Hansson >> Signed-off-by: Ulf Hansson >> --- >> include/linux/pm_domain.h | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/include/linux/pm_domain.h b/include/linux/pm_domain.h >> index 9206a4fef9ac..27fca748344a 100644 >> --- a/include/linux/pm_domain.h >> +++ b/include/linux/pm_domain.h >> @@ -44,6 +44,7 @@ struct genpd_power_state { >> s64 residency_ns; >> struct fwnode_handle *fwnode; >> ktime_t idle_time; >> + void *data; >> }; >> >> struct genpd_lock_ops; >> -- >> 2.17.1 >> Kind regards Uffe From mboxrd@z Thu Jan 1 00:00:00 1970 From: ulf.hansson@linaro.org (Ulf Hansson) Date: Mon, 25 Jun 2018 10:34:38 +0200 Subject: [PATCH v8 03/26] PM / Domains: Add generic data pointer to genpd_power_state struct In-Reply-To: References: <20180620172226.15012-1-ulf.hansson@linaro.org> <20180620172226.15012-4-ulf.hansson@linaro.org> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 24 June 2018 at 23:09, Rafael J. Wysocki wrote: > On Wed, Jun 20, 2018 at 7:22 PM, Ulf Hansson wrote: >> From: Lina Iyer >> >> Let's add a data pointer to the genpd_power_state struct, to allow >> platforms to store per state specific data. > > Can you please fold it into a patch actually using this pointer? Yep, no problem. Anyway, the change that uses the pointer is "[PATCH v8 21/26] drivers: firmware: psci: Add support for PM domains using genpd". Let's see if there is further comments, if not - perhaps you can squash $subject patch into that change? > >> Cc: Lina Iyer >> Signed-off-by: Lina Iyer >> Co-developed-by: Ulf Hansson >> Signed-off-by: Ulf Hansson >> --- >> include/linux/pm_domain.h | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/include/linux/pm_domain.h b/include/linux/pm_domain.h >> index 9206a4fef9ac..27fca748344a 100644 >> --- a/include/linux/pm_domain.h >> +++ b/include/linux/pm_domain.h >> @@ -44,6 +44,7 @@ struct genpd_power_state { >> s64 residency_ns; >> struct fwnode_handle *fwnode; >> ktime_t idle_time; >> + void *data; >> }; >> >> struct genpd_lock_ops; >> -- >> 2.17.1 >> Kind regards Uffe