* [PATCH 2/3] Documentation: add DT bindings for ARM SCPI power domains [not found] <1465228439-13457-1-git-send-email-sudeep.holla@arm.com> @ 2016-06-06 15:53 ` Sudeep Holla 2016-06-07 13:22 ` Mark Rutland 0 siblings, 1 reply; 4+ messages in thread From: Sudeep Holla @ 2016-06-06 15:53 UTC (permalink / raw) To: linux-kernel Cc: Sudeep Holla, Jon Medhurst, Mathieu Poirier, Suzuki K Poulose, Rob Herring, Mark Rutland, linux-arm-kernel, devicetree The System Control Processor (SCP) provides peripheral devices with power domains that can be enabled and disabled viathe System Control and Power Interface (SCPI) Message Protocol. Add bindings to allow probing of these device power domians. Cc: Rob Herring <robh+dt@kernel.org> Cc: Mark Rutland <mark.rutland@arm.com> Cc: linux-arm-kernel@lists.infradead.org Cc: devicetree@vger.kernel.org Signed-off-by: Sudeep Holla <sudeep.holla@arm.com> --- Documentation/devicetree/bindings/arm/arm,scpi.txt | 34 ++++++++++++++++++++++ 1 file changed, 34 insertions(+) diff --git a/Documentation/devicetree/bindings/arm/arm,scpi.txt b/Documentation/devicetree/bindings/arm/arm,scpi.txt index 313dabdc14f9..7141670d649b 100644 --- a/Documentation/devicetree/bindings/arm/arm,scpi.txt +++ b/Documentation/devicetree/bindings/arm/arm,scpi.txt @@ -87,10 +87,33 @@ SCPI provides an API to access the various sensors on the SoC. implementation for the IDs to use. For Juno R0 and Juno R1 refer to [3]. +Power domain bindings for the power domains based on SCPI Message Protocol +------------------------------------------------------------ + +This binding uses the generic power domain binding[4]. + +PM domain providers +=================== + +Required properties: + - #power-domain-cells : Should be 1. Contains the device or the power + domain ID value used by SCPI commands. + - num-domains: Total number of power domains provided by SCPI. This is + needed as the SCPI message protocol lacks a mechanism to + query this information runtime. + +PM domain consumers +=================== + +Required properties: + - power-domains : A phandle and PM domain specifier as defined by bindings of + the power controller specified by phandle. + [0] http://infocenter.arm.com/help/topic/com.arm.doc.dui0922b/index.html [1] Documentation/devicetree/bindings/clock/clock-bindings.txt [2] Documentation/devicetree/bindings/thermal/thermal.txt [3] http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.dui0922b/apas03s22.html +[4] Documentation/devicetree/bindings/power/power_domain.txt Example: @@ -144,6 +167,12 @@ scpi_protocol: scpi@2e000000 { compatible = "arm,scpi-sensors"; #thermal-sensor-cells = <1>; }; + + scpi_devpd: scpi-power-domains { + compatible = "arm,scpi-power-domains"; + num-domains = <2>; + #power-domain-cells = <1>; + }; }; cpu@0 { @@ -156,6 +185,7 @@ hdlcd@7ff60000 { ... reg = <0 0x7ff60000 0 0x1000>; clocks = <&scpi_clk 4>; + power-domains = <&scpi_devpd 1>; }; thermal-zones { @@ -186,3 +216,7 @@ The thermal-sensors property in the soc_thermal node uses the temperature sensor provided by SCP firmware to setup a thermal zone. The ID "3" is the sensor identifier for the temperature sensor as used by the firmware. + +The num-domains property in scpi-power-domains domain specifies that +SCPI provides 2 power domains. The hdlcd node uses the power domain with +domain ID 1. ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 2/3] Documentation: add DT bindings for ARM SCPI power domains 2016-06-06 15:53 ` [PATCH 2/3] Documentation: add DT bindings for ARM SCPI power domains Sudeep Holla @ 2016-06-07 13:22 ` Mark Rutland 2016-06-07 13:39 ` Sudeep Holla 0 siblings, 1 reply; 4+ messages in thread From: Mark Rutland @ 2016-06-07 13:22 UTC (permalink / raw) To: Sudeep Holla Cc: linux-kernel, Jon Medhurst, Mathieu Poirier, Suzuki K Poulose, Rob Herring, linux-arm-kernel, devicetree On Mon, Jun 06, 2016 at 04:53:58PM +0100, Sudeep Holla wrote: > The System Control Processor (SCP) provides peripheral devices with > power domains that can be enabled and disabled viathe System Control > and Power Interface (SCPI) Message Protocol. Add bindings to allow > probing of these device power domians. > > Cc: Rob Herring <robh+dt@kernel.org> > Cc: Mark Rutland <mark.rutland@arm.com> > Cc: linux-arm-kernel@lists.infradead.org > Cc: devicetree@vger.kernel.org > Signed-off-by: Sudeep Holla <sudeep.holla@arm.com> > --- > Documentation/devicetree/bindings/arm/arm,scpi.txt | 34 ++++++++++++++++++++++ > 1 file changed, 34 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/arm,scpi.txt b/Documentation/devicetree/bindings/arm/arm,scpi.txt > index 313dabdc14f9..7141670d649b 100644 > --- a/Documentation/devicetree/bindings/arm/arm,scpi.txt > +++ b/Documentation/devicetree/bindings/arm/arm,scpi.txt > @@ -87,10 +87,33 @@ SCPI provides an API to access the various sensors on the SoC. > implementation for the IDs to use. For Juno > R0 and Juno R1 refer to [3]. > > +Power domain bindings for the power domains based on SCPI Message Protocol > +------------------------------------------------------------ > + > +This binding uses the generic power domain binding[4]. > + > +PM domain providers > +=================== > + > +Required properties: > + - #power-domain-cells : Should be 1. Contains the device or the power > + domain ID value used by SCPI commands. > + - num-domains: Total number of power domains provided by SCPI. This is > + needed as the SCPI message protocol lacks a mechanism to > + query this information runtime. ^ I guess there should be an 'at' here. Are domain IDs zero-based and definitely non-sparse? What exactly does this matter for? Just for validation at parsing time, or is this strictly required for correctness? If we send a command with an invalid domain ID, would the FW reliably report an error that we can recover from? Otherwise, this looks ok. I'd just like to make sure I've understood correctly. Thanks, Mark. > + > +PM domain consumers > +=================== > + > +Required properties: > + - power-domains : A phandle and PM domain specifier as defined by bindings of > + the power controller specified by phandle. > + > [0] http://infocenter.arm.com/help/topic/com.arm.doc.dui0922b/index.html > [1] Documentation/devicetree/bindings/clock/clock-bindings.txt > [2] Documentation/devicetree/bindings/thermal/thermal.txt > [3] http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.dui0922b/apas03s22.html > +[4] Documentation/devicetree/bindings/power/power_domain.txt > > Example: > > @@ -144,6 +167,12 @@ scpi_protocol: scpi@2e000000 { > compatible = "arm,scpi-sensors"; > #thermal-sensor-cells = <1>; > }; > + > + scpi_devpd: scpi-power-domains { > + compatible = "arm,scpi-power-domains"; > + num-domains = <2>; > + #power-domain-cells = <1>; > + }; > }; > > cpu@0 { > @@ -156,6 +185,7 @@ hdlcd@7ff60000 { > ... > reg = <0 0x7ff60000 0 0x1000>; > clocks = <&scpi_clk 4>; > + power-domains = <&scpi_devpd 1>; > }; > > thermal-zones { > @@ -186,3 +216,7 @@ The thermal-sensors property in the soc_thermal node uses the > temperature sensor provided by SCP firmware to setup a thermal > zone. The ID "3" is the sensor identifier for the temperature sensor > as used by the firmware. > + > +The num-domains property in scpi-power-domains domain specifies that > +SCPI provides 2 power domains. The hdlcd node uses the power domain with > +domain ID 1. > -- > 2.7.4 > ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 2/3] Documentation: add DT bindings for ARM SCPI power domains 2016-06-07 13:22 ` Mark Rutland @ 2016-06-07 13:39 ` Sudeep Holla [not found] ` <5756CE8D.6040704-5wv7dgnIgG8@public.gmane.org> 0 siblings, 1 reply; 4+ messages in thread From: Sudeep Holla @ 2016-06-07 13:39 UTC (permalink / raw) To: Mark Rutland Cc: Sudeep Holla, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Jon Medhurst, Mathieu Poirier, Suzuki K Poulose, Rob Herring, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, devicetree-u79uwXL29TY76Z2rM5mHXA On 07/06/16 14:22, Mark Rutland wrote: > On Mon, Jun 06, 2016 at 04:53:58PM +0100, Sudeep Holla wrote: >> The System Control Processor (SCP) provides peripheral devices with >> power domains that can be enabled and disabled viathe System Control >> and Power Interface (SCPI) Message Protocol. Add bindings to allow >> probing of these device power domians. >> >> Cc: Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> >> Cc: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org> >> Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org >> Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org >> Signed-off-by: Sudeep Holla <sudeep.holla-5wv7dgnIgG8@public.gmane.org> >> --- >> Documentation/devicetree/bindings/arm/arm,scpi.txt | 34 ++++++++++++++++++++++ >> 1 file changed, 34 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/arm/arm,scpi.txt b/Documentation/devicetree/bindings/arm/arm,scpi.txt >> index 313dabdc14f9..7141670d649b 100644 >> --- a/Documentation/devicetree/bindings/arm/arm,scpi.txt >> +++ b/Documentation/devicetree/bindings/arm/arm,scpi.txt >> @@ -87,10 +87,33 @@ SCPI provides an API to access the various sensors on the SoC. >> implementation for the IDs to use. For Juno >> R0 and Juno R1 refer to [3]. >> >> +Power domain bindings for the power domains based on SCPI Message Protocol >> +------------------------------------------------------------ >> + >> +This binding uses the generic power domain binding[4]. >> + >> +PM domain providers >> +=================== >> + >> +Required properties: >> + - #power-domain-cells : Should be 1. Contains the device or the power >> + domain ID value used by SCPI commands. >> + - num-domains: Total number of power domains provided by SCPI. This is >> + needed as the SCPI message protocol lacks a mechanism to >> + query this information runtime. > ^ > I guess there should be an 'at' here. > Will fix. > Are domain IDs zero-based and definitely non-sparse? > Yes > What exactly does this matter for? Just for validation at parsing time, > or is this strictly required for correctness? > This is mainly to know the maximum number of power domains that firmware supports. This will help the software handling the provider part to setup the information in advance before any consumer request for the service. > If we send a command with an invalid domain ID, would the FW reliably > report an error that we can recover from? > Yes for anything above this value, firmware returns invalid parameter error. It's would be good to have that as a separate command instead of getting it via DT. We already have that for OPPs and clocks. Just this lacks that feature. > Otherwise, this looks ok. I'd just like to make sure I've understood > correctly. > Sure, thanks for the review. -- Regards, Sudeep -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 4+ messages in thread
[parent not found: <5756CE8D.6040704-5wv7dgnIgG8@public.gmane.org>]
* Re: [PATCH 2/3] Documentation: add DT bindings for ARM SCPI power domains [not found] ` <5756CE8D.6040704-5wv7dgnIgG8@public.gmane.org> @ 2016-06-07 14:45 ` Mark Rutland 0 siblings, 0 replies; 4+ messages in thread From: Mark Rutland @ 2016-06-07 14:45 UTC (permalink / raw) To: Sudeep Holla Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA, Jon Medhurst, Mathieu Poirier, Suzuki K Poulose, Rob Herring, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, devicetree-u79uwXL29TY76Z2rM5mHXA On Tue, Jun 07, 2016 at 02:39:25PM +0100, Sudeep Holla wrote: > > > On 07/06/16 14:22, Mark Rutland wrote: > >On Mon, Jun 06, 2016 at 04:53:58PM +0100, Sudeep Holla wrote: > >>The System Control Processor (SCP) provides peripheral devices with > >>power domains that can be enabled and disabled viathe System Control > >>and Power Interface (SCPI) Message Protocol. Add bindings to allow > >>probing of these device power domians. > >> > >>Cc: Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> > >>Cc: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org> > >>Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org > >>Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > >>Signed-off-by: Sudeep Holla <sudeep.holla-5wv7dgnIgG8@public.gmane.org> > >>--- > >> Documentation/devicetree/bindings/arm/arm,scpi.txt | 34 ++++++++++++++++++++++ > >> 1 file changed, 34 insertions(+) > >> > >>diff --git a/Documentation/devicetree/bindings/arm/arm,scpi.txt b/Documentation/devicetree/bindings/arm/arm,scpi.txt > >>index 313dabdc14f9..7141670d649b 100644 > >>--- a/Documentation/devicetree/bindings/arm/arm,scpi.txt > >>+++ b/Documentation/devicetree/bindings/arm/arm,scpi.txt > >>@@ -87,10 +87,33 @@ SCPI provides an API to access the various sensors on the SoC. > >> implementation for the IDs to use. For Juno > >> R0 and Juno R1 refer to [3]. > >> > >>+Power domain bindings for the power domains based on SCPI Message Protocol > >>+------------------------------------------------------------ > >>+ > >>+This binding uses the generic power domain binding[4]. > >>+ > >>+PM domain providers > >>+=================== > >>+ > >>+Required properties: > >>+ - #power-domain-cells : Should be 1. Contains the device or the power > >>+ domain ID value used by SCPI commands. > >>+ - num-domains: Total number of power domains provided by SCPI. This is > >>+ needed as the SCPI message protocol lacks a mechanism to > >>+ query this information runtime. > > ^ > >I guess there should be an 'at' here. > > > > Will fix. > > >Are domain IDs zero-based and definitely non-sparse? > > > > Yes Ok. So FWIW, with the fix above: Acked-by: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org> Thanks, Mark. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-06-07 14:45 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <1465228439-13457-1-git-send-email-sudeep.holla@arm.com> 2016-06-06 15:53 ` [PATCH 2/3] Documentation: add DT bindings for ARM SCPI power domains Sudeep Holla 2016-06-07 13:22 ` Mark Rutland 2016-06-07 13:39 ` Sudeep Holla [not found] ` <5756CE8D.6040704-5wv7dgnIgG8@public.gmane.org> 2016-06-07 14:45 ` Mark Rutland
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).