arm64: dts: qcom: sm8350: move DPU opp-table to its node
authorKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Tue, 7 Nov 2023 10:35:40 +0000 (11:35 +0100)
committerBjorn Andersson <andersson@kernel.org>
Thu, 7 Dec 2023 16:15:06 +0000 (08:15 -0800)
The Qualcomm MDSS bindings expect that DPU opp-table is defined within
DPU node:

  sm8350-hdk.dtb: display-subsystem@ae00000: Unevaluated properties are not allowed ('opp-table' was unexpected)

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Link: https://lore.kernel.org/r/20231107103540.27353-3-krzysztof.kozlowski@linaro.org
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
arch/arm64/boot/dts/qcom/sm8350.dtsi

index 94dcc4b6410521494faa5c88e61631cbcdc4b360..2055b748e4c903eed36f6caaea8e2f3c1fd2fdc4 100644 (file)
                        #size-cells = <2>;
                        ranges;
 
-                       dpu_opp_table: opp-table {
-                               compatible = "operating-points-v2";
-
-                               /* TODO: opp-200000000 should work with
-                                * &rpmhpd_opp_low_svs, but one some of
-                                * sm8350_hdk boards reboot using this
-                                * opp.
-                                */
-                               opp-200000000 {
-                                       opp-hz = /bits/ 64 <200000000>;
-                                       required-opps = <&rpmhpd_opp_svs>;
-                               };
-
-                               opp-300000000 {
-                                       opp-hz = /bits/ 64 <300000000>;
-                                       required-opps = <&rpmhpd_opp_svs>;
-                               };
-
-                               opp-345000000 {
-                                       opp-hz = /bits/ 64 <345000000>;
-                                       required-opps = <&rpmhpd_opp_svs_l1>;
-                               };
-
-                               opp-460000000 {
-                                       opp-hz = /bits/ 64 <460000000>;
-                                       required-opps = <&rpmhpd_opp_nom>;
-                               };
-                       };
-
                        mdss_mdp: display-controller@ae01000 {
                                compatible = "qcom,sm8350-dpu";
                                reg = <0 0x0ae01000 0 0x8f000>,
                                interrupt-parent = <&mdss>;
                                interrupts = <0>;
 
+                               dpu_opp_table: opp-table {
+                                       compatible = "operating-points-v2";
+
+                                       /* TODO: opp-200000000 should work with
+                                        * &rpmhpd_opp_low_svs, but one some of
+                                        * sm8350_hdk boards reboot using this
+                                        * opp.
+                                        */
+                                       opp-200000000 {
+                                               opp-hz = /bits/ 64 <200000000>;
+                                               required-opps = <&rpmhpd_opp_svs>;
+                                       };
+
+                                       opp-300000000 {
+                                               opp-hz = /bits/ 64 <300000000>;
+                                               required-opps = <&rpmhpd_opp_svs>;
+                                       };
+
+                                       opp-345000000 {
+                                               opp-hz = /bits/ 64 <345000000>;
+                                               required-opps = <&rpmhpd_opp_svs_l1>;
+                                       };
+
+                                       opp-460000000 {
+                                               opp-hz = /bits/ 64 <460000000>;
+                                               required-opps = <&rpmhpd_opp_nom>;
+                                       };
+                               };
+
                                ports {
                                        #address-cells = <1>;
                                        #size-cells = <0>;