On 14/10/2020 03:46, Stephen Boyd wrote:
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.
It is to be able to resolve "power-domains" and "required-opps"
properties values.
+ 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 {
--
With best wishes
Dmitry
