| Message ID | 1475699519-109623-6-git-send-email-lina.iyer@linaro.org |
|---|---|
| State | New |
| Headers | show |
On Thu, Oct 06 2016 at 02:09 -0600, Geert Uytterhoeven wrote: >On Thu, Oct 6, 2016 at 10:06 AM, Geert Uytterhoeven ><geert@linux-m68k.org> wrote: >> On Wed, Oct 5, 2016 at 10:31 PM, Lina Iyer <lina.iyer@linaro.org> wrote: >>> Update DT bindings to describe idle states of PM domains. >>> >>> This patch is based on the original patch by Marc Titinger. >>> >>> Cc: <devicetree@vger.kernel.org> >>> Signed-off-by: Marc Titinger <mtitinger+renesas@baylibre.com> >>> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> >>> Signed-off-by: Lina Iyer <lina.iyer@linaro.org> >>> Acked-by: Rob Herring <robh@kernel.org> >>> --- >>> .../devicetree/bindings/power/power_domain.txt | 36 ++++++++++++++++++++++ >>> 1 file changed, 36 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/power/power_domain.txt b/Documentation/devicetree/bindings/power/power_domain.txt >>> index 025b5e7..a043315 100644 >>> --- a/Documentation/devicetree/bindings/power/power_domain.txt >>> +++ b/Documentation/devicetree/bindings/power/power_domain.txt >>> @@ -29,6 +29,10 @@ Optional properties: >>> specified by this binding. More details about power domain specifier are >>> available in the next section. >>> >>> +- domain-idle-states : A phandle of an idle-state that shall be soaked into a >>> + generic domain power state. The idle state definitions are >>> + compatible with arm,idle-state specified in [1]. >>> + >>> Example: >>> >>> power: power-controller@12340000 { >>> @@ -59,6 +63,36 @@ The nodes above define two power controllers: 'parent' and 'child'. >>> Domains created by the 'child' power controller are subdomains of '0' power >>> domain provided by the 'parent' power controller. >>> >>> +Example 3: >>> + parent: power-controller@12340000 { >> >> With W=1, this is gonna trigger: >> >> Warning (unit_address_vs_reg): Node foo has a unit name, but no reg property >> >> Yes, there are pre-existing users in this file. > >Scrap this... switching desktops causes loss of position... > >>> + compatible = "foo,power-controller"; >>> + reg = <0x12340000 0x1000>; >>> + #power-domain-cells = <1>; >>> + domain-idle-states = <&DOMAIN_RET, &DOMAIN_PWR_DN>; >>> + }; > >+ DOMAIN_RET: state@0 { >+ compatible = "arm,idle-state"; >+ entry-latency-us = <1000>; >+ exit-latency-us = <2000>; >+ min-residency-us = <10000>; > >This one is gonna trigger > >Warning (unit_address_vs_reg): Node foo has a unit name, but no reg property > >+ }; > Its just an example, Will fix the example. >Gr{oetje,eeting}s, > > Geert > >-- >Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org > >In personal conversations with technical people, I call myself a hacker. But >when I'm talking to journalists I just say "programmer" or something like that. > -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/power/power_domain.txt b/Documentation/devicetree/bindings/power/power_domain.txt index 025b5e7..a043315 100644 --- a/Documentation/devicetree/bindings/power/power_domain.txt +++ b/Documentation/devicetree/bindings/power/power_domain.txt @@ -29,6 +29,10 @@ Optional properties: specified by this binding. More details about power domain specifier are available in the next section. +- domain-idle-states : A phandle of an idle-state that shall be soaked into a + generic domain power state. The idle state definitions are + compatible with arm,idle-state specified in [1]. + Example: power: power-controller@12340000 { @@ -59,6 +63,36 @@ The nodes above define two power controllers: 'parent' and 'child'. Domains created by the 'child' power controller are subdomains of '0' power domain provided by the 'parent' power controller. +Example 3: + parent: power-controller@12340000 { + compatible = "foo,power-controller"; + reg = <0x12340000 0x1000>; + #power-domain-cells = <1>; + domain-idle-states = <&DOMAIN_RET, &DOMAIN_PWR_DN>; + }; + + child: power-controller@12341000 { + compatible = "foo,power-controller"; + reg = <0x12341000 0x1000>; + power-domains = <&parent 0>; + #power-domain-cells = <1>; + domain-idle-states = <&DOMAIN_PWR_DN>; + }; + + DOMAIN_RET: state@0 { + compatible = "arm,idle-state"; + entry-latency-us = <1000>; + exit-latency-us = <2000>; + min-residency-us = <10000>; + }; + + DOMAIN_PWR_DN: state@1 { + compatible = "arm,idle-state"; + entry-latency-us = <5000>; + exit-latency-us = <8000>; + min-residency-us = <7000>; + }; + ==PM domain consumers== Required properties: @@ -76,3 +110,5 @@ Example: The node above defines a typical PM domain consumer device, which is located inside a PM domain with index 0 of a power controller represented by a node with the label "power". + +[1]. Documentation/devicetree/bindings/arm/idle-states.txt