From: Lina Iyer <lina.iyer@linaro.org> To: Mark Rutland <mark.rutland@arm.com> Cc: ulf.hansson@linaro.org, khilman@kernel.org, rjw@rjwysocki.net, linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, geert@linux-m68k.org, k.kozlowski@samsung.com, andy.gross@linaro.org, sboyd@codeaurora.org, linux-arm-msm@vger.kernel.org, lorenzo.pieralisi@arm.com, ahaslam@baylibre.com, mtitinger@baylibre.com, Axel Haslam <ahaslam+renesas@baylibre.com>, devicetree@vger.kernel.org, Marc Titinger <mtitinger+renesas@baylibre.com> Subject: Re: [PATCH 02/14] dt/bindings: update binding for PM domain idle states Date: Thu, 23 Jun 2016 12:04:51 -0600 [thread overview] Message-ID: <20160623180451.GD1115@linaro.org> (raw) In-Reply-To: <20160623173312.GA22204@leverpostej> Hi Mark, On Thu, Jun 23 2016 at 11:35 -0600, Mark Rutland wrote: >Hi, > >On Wed, Jun 22, 2016 at 01:36:37PM -0600, Lina Iyer wrote: >> From: Axel Haslam <ahaslam+renesas@baylibre.com> >> >> Update DT bindings to describe idle states of PM domains. >> >> Cc: <devicetree@vger.kernel.org> >> Signed-off-by: Marc Titinger <mtitinger+renesas@baylibre.com> >> Signed-off-by: Lina Iyer <lina.iyer@linaro.org> >> [Lina: Added state properties, removed state names, wakeup-latency, >> added of_pm_genpd_init() API, pruned commit text] >> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> >> [Ulf: Moved around code to make it compile properly, rebased on top of multiple state support] >> --- >> .../devicetree/bindings/power/power_domain.txt | 70 ++++++++++++++++++++++ >> 1 file changed, 70 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/power/power_domain.txt b/Documentation/devicetree/bindings/power/power_domain.txt >> index 025b5e7..41e8dda 100644 >> --- a/Documentation/devicetree/bindings/power/power_domain.txt >> +++ b/Documentation/devicetree/bindings/power/power_domain.txt >> @@ -29,6 +29,43 @@ Optional properties: >> specified by this binding. More details about power domain specifier are >> available in the next section. >> >> +- power-states : A phandle of an idle-state that shall be soaked into a >> + generic domain power state. > >It's somewhat unfortunate that this gives us two possible locations for >idle state lists (under the /cpus node and in a pm-domains node), >especially as it's not clear what would happen were a DT to have both. > >I would prefer that we extend the existing bindings such that states can >refer to the power domains which they affect. > I agree. The CPU idle states have become defined to be specific to CPUs. PM Domain idle states are generic for any type of domain. I am hoping at some point, we could converge and use the same idle state, but that would mean changing the CPU idle states to make it generic. At some point, during my development, I did use the arm,idle-state for domains as well, but the binding definitions were too restrictive for a generic PM domain. I would be willing to make the change to CPU idle states to make it generic and then we could just reference domain and CPU idle states using the same bindings. Are we okay with that, specifically, arm,psci-suspend-param? This binding is very restrictive in its description. What we pass to the platform driver upon choosing a domain state is very platform specific and therefore has to be generic in its description. Thanks, Lina >> +==Power state== >> + >> +A PM domain power state describes an idle state of a domain and must be >> +have the following properties - >> + >> + - entry-latency-us >> + Usage: Not required if wakeup-latency-us is provided. >> + Value type: <prop-encoded-array> >> + Definition: u32 value representing worst case latency in >> + microseconds required to enter the idle state. >> + The exit-latency-us duration may be guaranteed >> + only after entry-latency-us has passed. >> + >> + - exit-latency-us >> + Usage: Not required if wakeup-latency-us is provided. >> + Value type: <prop-encoded-array> >> + Definition: u32 value representing worst case latency >> + in microseconds required to exit the idle state. > >These are part of the existing idle state binding >(Documentation/devicetree/bindings/arm/idle-states.txt), and I would >prefer that we did not duplicate this nor come up with an independent >set of idle state bindings. > >> + >> + - residency-us: >> + Usage: Optional >> + Value type: <prop-encoded-array> >> + Definition: A u32 value representing the time for which a >> + domain must be idle in the state to reap power saving benefits >> + of entering the state. > >This is "min-residency-us" per the existing bindings. > >> + >> + - state-param: >> + Usage: Optional >> + Value type: <prop-encoded-array> >> + Definition: A u32 value as defined by the state. May be used >> + by the driver to hold state related u32 data. In the case of >> + PSCI, an extended state-id for the domain may be encoded in >> + this value. > >This is "arm,psci-suspend-param", per the existing bindings. > >Thanks, >Mark.
WARNING: multiple messages have this Message-ID (diff)
From: lina.iyer@linaro.org (Lina Iyer) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 02/14] dt/bindings: update binding for PM domain idle states Date: Thu, 23 Jun 2016 12:04:51 -0600 [thread overview] Message-ID: <20160623180451.GD1115@linaro.org> (raw) In-Reply-To: <20160623173312.GA22204@leverpostej> Hi Mark, On Thu, Jun 23 2016 at 11:35 -0600, Mark Rutland wrote: >Hi, > >On Wed, Jun 22, 2016 at 01:36:37PM -0600, Lina Iyer wrote: >> From: Axel Haslam <ahaslam+renesas@baylibre.com> >> >> Update DT bindings to describe idle states of PM domains. >> >> Cc: <devicetree@vger.kernel.org> >> Signed-off-by: Marc Titinger <mtitinger+renesas@baylibre.com> >> Signed-off-by: Lina Iyer <lina.iyer@linaro.org> >> [Lina: Added state properties, removed state names, wakeup-latency, >> added of_pm_genpd_init() API, pruned commit text] >> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> >> [Ulf: Moved around code to make it compile properly, rebased on top of multiple state support] >> --- >> .../devicetree/bindings/power/power_domain.txt | 70 ++++++++++++++++++++++ >> 1 file changed, 70 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/power/power_domain.txt b/Documentation/devicetree/bindings/power/power_domain.txt >> index 025b5e7..41e8dda 100644 >> --- a/Documentation/devicetree/bindings/power/power_domain.txt >> +++ b/Documentation/devicetree/bindings/power/power_domain.txt >> @@ -29,6 +29,43 @@ Optional properties: >> specified by this binding. More details about power domain specifier are >> available in the next section. >> >> +- power-states : A phandle of an idle-state that shall be soaked into a >> + generic domain power state. > >It's somewhat unfortunate that this gives us two possible locations for >idle state lists (under the /cpus node and in a pm-domains node), >especially as it's not clear what would happen were a DT to have both. > >I would prefer that we extend the existing bindings such that states can >refer to the power domains which they affect. > I agree. The CPU idle states have become defined to be specific to CPUs. PM Domain idle states are generic for any type of domain. I am hoping at some point, we could converge and use the same idle state, but that would mean changing the CPU idle states to make it generic. At some point, during my development, I did use the arm,idle-state for domains as well, but the binding definitions were too restrictive for a generic PM domain. I would be willing to make the change to CPU idle states to make it generic and then we could just reference domain and CPU idle states using the same bindings. Are we okay with that, specifically, arm,psci-suspend-param? This binding is very restrictive in its description. What we pass to the platform driver upon choosing a domain state is very platform specific and therefore has to be generic in its description. Thanks, Lina >> +==Power state== >> + >> +A PM domain power state describes an idle state of a domain and must be >> +have the following properties - >> + >> + - entry-latency-us >> + Usage: Not required if wakeup-latency-us is provided. >> + Value type: <prop-encoded-array> >> + Definition: u32 value representing worst case latency in >> + microseconds required to enter the idle state. >> + The exit-latency-us duration may be guaranteed >> + only after entry-latency-us has passed. >> + >> + - exit-latency-us >> + Usage: Not required if wakeup-latency-us is provided. >> + Value type: <prop-encoded-array> >> + Definition: u32 value representing worst case latency >> + in microseconds required to exit the idle state. > >These are part of the existing idle state binding >(Documentation/devicetree/bindings/arm/idle-states.txt), and I would >prefer that we did not duplicate this nor come up with an independent >set of idle state bindings. > >> + >> + - residency-us: >> + Usage: Optional >> + Value type: <prop-encoded-array> >> + Definition: A u32 value representing the time for which a >> + domain must be idle in the state to reap power saving benefits >> + of entering the state. > >This is "min-residency-us" per the existing bindings. > >> + >> + - state-param: >> + Usage: Optional >> + Value type: <prop-encoded-array> >> + Definition: A u32 value as defined by the state. May be used >> + by the driver to hold state related u32 data. In the case of >> + PSCI, an extended state-id for the domain may be encoded in >> + this value. > >This is "arm,psci-suspend-param", per the existing bindings. > >Thanks, >Mark.
next prev parent reply other threads:[~2016-06-23 18:04 UTC|newest] Thread overview: 63+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-06-22 19:36 [PATCH 00/14] PM: SoC idle support using PM domains Lina Iyer 2016-06-22 19:36 ` Lina Iyer 2016-06-22 19:36 ` [PATCH 01/14] PM / Domains: Allow domain power states to be read from DT Lina Iyer 2016-06-22 19:36 ` Lina Iyer 2016-06-23 17:38 ` Mark Rutland 2016-06-23 17:38 ` Mark Rutland [not found] ` <1466624209-27432-1-git-send-email-lina.iyer-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2016-06-22 19:36 ` [PATCH 02/14] dt/bindings: update binding for PM domain idle states Lina Iyer 2016-06-22 19:36 ` Lina Iyer [not found] ` <1466624209-27432-3-git-send-email-lina.iyer-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2016-06-23 17:35 ` Mark Rutland 2016-06-23 17:35 ` Mark Rutland 2016-06-23 18:04 ` Lina Iyer [this message] 2016-06-23 18:04 ` Lina Iyer [not found] ` <20160623180451.GD1115-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2016-06-23 18:19 ` Mark Rutland 2016-06-23 18:19 ` Mark Rutland 2016-06-23 18:39 ` Lina Iyer 2016-06-23 18:39 ` Lina Iyer 2016-07-27 11:11 ` [draft] " Brendan Jackman 2016-06-22 19:36 ` [PATCH 03/14] PM / Domains: Abstract genpd locking Lina Iyer 2016-06-22 19:36 ` Lina Iyer 2016-06-22 19:36 ` [PATCH 04/14] PM / Domains: Support IRQ safe PM domains Lina Iyer 2016-06-22 19:36 ` Lina Iyer 2016-06-22 19:36 ` [PATCH 05/14] PM / doc: update device documentation for devices in " Lina Iyer 2016-06-22 19:36 ` Lina Iyer 2016-07-25 22:50 ` Kevin Hilman 2016-07-25 22:50 ` Kevin Hilman 2016-07-25 23:21 ` Lina Iyer 2016-07-25 23:21 ` Lina Iyer 2016-06-22 19:36 ` [PATCH 06/14] PM / cpu_domains: Setup PM domains for CPUs/clusters Lina Iyer 2016-06-22 19:36 ` Lina Iyer 2016-06-22 19:36 ` [PATCH 07/14] ARM: cpuidle: Add runtime PM support for CPUs Lina Iyer 2016-06-22 19:36 ` Lina Iyer 2016-06-22 19:36 ` [PATCH 08/14] timer: Export next wake up of a CPU Lina Iyer 2016-06-22 19:36 ` Lina Iyer 2016-07-26 18:26 ` Kevin Hilman 2016-07-26 18:26 ` Kevin Hilman 2016-07-27 9:14 ` Thomas Gleixner 2016-07-27 9:14 ` Thomas Gleixner 2016-07-27 15:04 ` Lina Iyer 2016-07-27 15:04 ` Lina Iyer 2016-06-22 19:36 ` [PATCH 09/14] PM / cpu_domains: Add PM Domain governor for CPUs Lina Iyer 2016-06-22 19:36 ` Lina Iyer 2016-06-22 19:36 ` [PATCH 10/14] doc / cpu_domains: Describe CPU PM domains setup and governor Lina Iyer 2016-06-22 19:36 ` Lina Iyer 2016-06-22 19:36 ` [PATCH 11/14] drivers: firmware: psci: Allow OS Initiated suspend mode Lina Iyer 2016-06-22 19:36 ` Lina Iyer 2016-06-24 4:25 ` Vikas Sajjan 2016-06-24 4:25 ` Vikas Sajjan 2016-06-24 16:53 ` Lina Iyer 2016-06-24 16:53 ` Lina Iyer 2016-06-27 10:12 ` Mark Rutland 2016-06-27 10:12 ` Mark Rutland 2016-06-28 6:07 ` Vikas Sajjan 2016-06-28 6:07 ` Vikas Sajjan 2016-06-22 19:36 ` [PATCH 12/14] drivers: firmware: psci: Support cluster idle states for OS-Initiated Lina Iyer 2016-06-22 19:36 ` Lina Iyer 2016-06-22 19:36 ` [PATCH 13/14] ARM64: dts: Add PSCI cpuidle support for MSM8916 Lina Iyer 2016-06-22 19:36 ` Lina Iyer 2016-06-22 19:36 ` [PATCH 14/14] ARM64: dts: Define CPU power domain " Lina Iyer 2016-06-22 19:36 ` Lina Iyer 2016-07-26 22:08 ` [PATCH 00/14] PM: SoC idle support using PM domains Kevin Hilman 2016-07-26 22:08 ` Kevin Hilman 2016-07-27 15:06 ` Lina Iyer 2016-07-27 15:06 ` Lina Iyer
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20160623180451.GD1115@linaro.org \ --to=lina.iyer@linaro.org \ --cc=ahaslam+renesas@baylibre.com \ --cc=ahaslam@baylibre.com \ --cc=andy.gross@linaro.org \ --cc=devicetree@vger.kernel.org \ --cc=geert@linux-m68k.org \ --cc=k.kozlowski@samsung.com \ --cc=khilman@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-arm-msm@vger.kernel.org \ --cc=linux-pm@vger.kernel.org \ --cc=lorenzo.pieralisi@arm.com \ --cc=mark.rutland@arm.com \ --cc=mtitinger+renesas@baylibre.com \ --cc=mtitinger@baylibre.com \ --cc=rjw@rjwysocki.net \ --cc=sboyd@codeaurora.org \ --cc=ulf.hansson@linaro.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.