Quoting Dmitry Baryshkov (2020-10-05 15:59:12) > SM8250 requires special power domain for accessing MMDS_GDSC registers. Heh, not sure it's special. > Add bindings for the MMCX power domain. > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > Reviewed-by: Rob Herring <robh@xxxxxxxxxx> > --- > .../bindings/clock/qcom,sdm845-dispcc.yaml | 28 +++++++++++++++++++ > 1 file changed, 28 insertions(+) > > diff --git a/Documentation/devicetree/bindings/clock/qcom,sdm845-dispcc.yaml b/Documentation/devicetree/bindings/clock/qcom,sdm845-dispcc.yaml > index 4a3be733d042..ff0db55470ac 100644 > --- a/Documentation/devicetree/bindings/clock/qcom,sdm845-dispcc.yaml > +++ b/Documentation/devicetree/bindings/clock/qcom,sdm845-dispcc.yaml > @@ -97,5 +108,22 @@ examples: > #clock-cells = <1>; > #reset-cells = <1>; > #power-domain-cells = <1>; > + /* this is a part of sm8250 setup the power domain example */ > + power-domains = <&rpmhpd SDM845_CX>; > + power-domain-names = "mmcx"; > + required-opps = <&rpmhpd_opp_low_svs>; > + }; > + rpmhpd: power-controller { Do we need this node in the example? I think it isn't required but I guess it's OK. > + compatible = "qcom,sdm845-rpmhpd"; > + #power-domain-cells = <1>; > + operating-points-v2 = <&rpmhpd_opp_table>; > + > + rpmhpd_opp_table: opp-table { > + compatible = "operating-points-v2"; > + > + rpmhpd_opp_low_svs: opp3 {