arm64: dts: qcom: sc7180: move QUP and QSPI opp tables out of SoC node
authorKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Sat, 10 Dec 2022 11:57:01 +0000 (12:57 +0100)
committerBjorn Andersson <andersson@kernel.org>
Wed, 28 Dec 2022 04:05:58 +0000 (22:05 -0600)
The SoC node is a simple-bus and its schema expect to have nodes only
with unit addresses:

  sc7180-trogdor-lazor-r3.dtb: soc@0: opp-table-qspi: {'compatible': ['operating-points-v2'], 'phandle': [[186]], 'opp-75000000':
    ...  'required-opps': [[47]]}} should not be valid under {'type': 'object'}

Move to top-level OPP tables:
 - QUP which is shared between multiple nodes,
 - QSPI which cannot be placed in its node due to address/size cells.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
Signed-off-by: Bjorn Andersson <andersson@kernel.org>
Link: https://lore.kernel.org/r/20221210115704.97614-1-krzysztof.kozlowski@linaro.org
arch/arm64/boot/dts/qcom/sc7180.dtsi

index 23f5920..94b35bb 100644 (file)
                };
        };
 
+       qspi_opp_table: opp-table-qspi {
+               compatible = "operating-points-v2";
+
+               opp-75000000 {
+                       opp-hz = /bits/ 64 <75000000>;
+                       required-opps = <&rpmhpd_opp_low_svs>;
+               };
+
+               opp-150000000 {
+                       opp-hz = /bits/ 64 <150000000>;
+                       required-opps = <&rpmhpd_opp_svs>;
+               };
+
+               opp-300000000 {
+                       opp-hz = /bits/ 64 <300000000>;
+                       required-opps = <&rpmhpd_opp_nom>;
+               };
+       };
+
+       qup_opp_table: opp-table-qup {
+               compatible = "operating-points-v2";
+
+               opp-75000000 {
+                       opp-hz = /bits/ 64 <75000000>;
+                       required-opps = <&rpmhpd_opp_low_svs>;
+               };
+
+               opp-100000000 {
+                       opp-hz = /bits/ 64 <100000000>;
+                       required-opps = <&rpmhpd_opp_svs>;
+               };
+
+               opp-128000000 {
+                       opp-hz = /bits/ 64 <128000000>;
+                       required-opps = <&rpmhpd_opp_nom>;
+               };
+       };
+
        memory@80000000 {
                device_type = "memory";
                /* We expect the bootloader to fill in the size */
                        };
                };
 
-               qup_opp_table: opp-table-qup {
-                       compatible = "operating-points-v2";
-
-                       opp-75000000 {
-                               opp-hz = /bits/ 64 <75000000>;
-                               required-opps = <&rpmhpd_opp_low_svs>;
-                       };
-
-                       opp-100000000 {
-                               opp-hz = /bits/ 64 <100000000>;
-                               required-opps = <&rpmhpd_opp_svs>;
-                       };
-
-                       opp-128000000 {
-                               opp-hz = /bits/ 64 <128000000>;
-                               required-opps = <&rpmhpd_opp_nom>;
-                       };
-               };
-
                qupv3_id_0: geniqup@8c0000 {
                        compatible = "qcom,geni-se-qup";
                        reg = <0 0x008c0000 0 0x6000>;
                        };
                };
 
-               qspi_opp_table: opp-table-qspi {
-                       compatible = "operating-points-v2";
-
-                       opp-75000000 {
-                               opp-hz = /bits/ 64 <75000000>;
-                               required-opps = <&rpmhpd_opp_low_svs>;
-                       };
-
-                       opp-150000000 {
-                               opp-hz = /bits/ 64 <150000000>;
-                               required-opps = <&rpmhpd_opp_svs>;
-                       };
-
-                       opp-300000000 {
-                               opp-hz = /bits/ 64 <300000000>;
-                               required-opps = <&rpmhpd_opp_nom>;
-                       };
-               };
-
                qspi: spi@88dc000 {
                        compatible = "qcom,sc7180-qspi", "qcom,qspi-v1";
                        reg = <0 0x088dc000 0 0x600>;