| Message ID | 20190823081703.17325-4-mkshah@codeaurora.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Add RSC power domain support | expand |
On Fri, Aug 23, 2019 at 01:47:00PM +0530, Maulik Shah wrote: > In addition to transmitting resource state requests to the remote > processor, the RSC is responsible for powering off/lowering the > requirements from CPUs subsystem for the associated hardware like > buses, clocks, and regulators when all CPUs and cluster is powered down. > > The power domain is configured to a low power state and when all the > CPUs are powered down, the RSC can lower resource state requirements > and power down the rails that power the CPUs. > > Add PM domain specifier property for RSC controller. > > Cc: devicetree@vger.kernel.org > Signed-off-by: Maulik Shah <mkshah@codeaurora.org> > Reviewed-by: Stephen Boyd <swboyd@chromium.org> > --- > Documentation/devicetree/bindings/soc/qcom/rpmh-rsc.txt | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/Documentation/devicetree/bindings/soc/qcom/rpmh-rsc.txt b/Documentation/devicetree/bindings/soc/qcom/rpmh-rsc.txt > index 9b86d1eff219..d0ab6e9b6745 100644 > --- a/Documentation/devicetree/bindings/soc/qcom/rpmh-rsc.txt > +++ b/Documentation/devicetree/bindings/soc/qcom/rpmh-rsc.txt > @@ -83,6 +83,13 @@ Properties: > Value type: <string> > Definition: Name for the RSC. The name would be used in trace logs. > > +- #power-domain-cells: > + Usage: optional > + Value type: <u32> > + Definition: Number of cells in power domain specifier. Optional for > + controllers that may be in 'solver' state where they can > + be in autonomous mode executing low power modes. What's the value? It's always 0? > + > Drivers that want to use the RSC to communicate with RPMH must specify their > bindings as child nodes of the RSC controllers they wish to communicate with. > > @@ -112,6 +119,7 @@ TCS-OFFSET: 0xD00 > <SLEEP_TCS 3>, > <WAKE_TCS 3>, > <CONTROL_TCS 1>; > + #power-domain-cells = <0>; > }; > > Example 2: > -- > QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation. >
On 8/28/2019 4:02 AM, Rob Herring wrote: > On Fri, Aug 23, 2019 at 01:47:00PM +0530, Maulik Shah wrote: >> In addition to transmitting resource state requests to the remote >> processor, the RSC is responsible for powering off/lowering the >> requirements from CPUs subsystem for the associated hardware like >> buses, clocks, and regulators when all CPUs and cluster is powered down. >> >> The power domain is configured to a low power state and when all the >> CPUs are powered down, the RSC can lower resource state requirements >> and power down the rails that power the CPUs. >> >> Add PM domain specifier property for RSC controller. >> >> Cc: devicetree@vger.kernel.org >> Signed-off-by: Maulik Shah <mkshah@codeaurora.org> >> Reviewed-by: Stephen Boyd <swboyd@chromium.org> >> --- >> Documentation/devicetree/bindings/soc/qcom/rpmh-rsc.txt | 8 ++++++++ >> 1 file changed, 8 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/soc/qcom/rpmh-rsc.txt b/Documentation/devicetree/bindings/soc/qcom/rpmh-rsc.txt >> index 9b86d1eff219..d0ab6e9b6745 100644 >> --- a/Documentation/devicetree/bindings/soc/qcom/rpmh-rsc.txt >> +++ b/Documentation/devicetree/bindings/soc/qcom/rpmh-rsc.txt >> @@ -83,6 +83,13 @@ Properties: >> Value type: <string> >> Definition: Name for the RSC. The name would be used in trace logs. >> >> +- #power-domain-cells: >> + Usage: optional >> + Value type: <u32> >> + Definition: Number of cells in power domain specifier. Optional for >> + controllers that may be in 'solver' state where they can >> + be in autonomous mode executing low power modes. > What's the value? It's always 0? yes. its value is always 0. i will update definition to mention this in next version. >> + >> Drivers that want to use the RSC to communicate with RPMH must specify their >> bindings as child nodes of the RSC controllers they wish to communicate with. >> >> @@ -112,6 +119,7 @@ TCS-OFFSET: 0xD00 >> <SLEEP_TCS 3>, >> <WAKE_TCS 3>, >> <CONTROL_TCS 1>; >> + #power-domain-cells = <0>; >> }; >> >> Example 2: >> -- >> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation. >>
diff --git a/Documentation/devicetree/bindings/soc/qcom/rpmh-rsc.txt b/Documentation/devicetree/bindings/soc/qcom/rpmh-rsc.txt index 9b86d1eff219..d0ab6e9b6745 100644 --- a/Documentation/devicetree/bindings/soc/qcom/rpmh-rsc.txt +++ b/Documentation/devicetree/bindings/soc/qcom/rpmh-rsc.txt @@ -83,6 +83,13 @@ Properties: Value type: <string> Definition: Name for the RSC. The name would be used in trace logs. +- #power-domain-cells: + Usage: optional + Value type: <u32> + Definition: Number of cells in power domain specifier. Optional for + controllers that may be in 'solver' state where they can + be in autonomous mode executing low power modes. + Drivers that want to use the RSC to communicate with RPMH must specify their bindings as child nodes of the RSC controllers they wish to communicate with. @@ -112,6 +119,7 @@ TCS-OFFSET: 0xD00 <SLEEP_TCS 3>, <WAKE_TCS 3>, <CONTROL_TCS 1>; + #power-domain-cells = <0>; }; Example 2: