arm64: dts: qcom: msm8996: Move '#clock-cells' to QMP PHY child node
authorShawn Guo <shawn.guo@linaro.org>
Wed, 29 Sep 2021 03:42:46 +0000 (11:42 +0800)
committerBjorn Andersson <bjorn.andersson@linaro.org>
Sun, 24 Oct 2021 18:04:03 +0000 (13:04 -0500)
'#clock-cells' is a required property of QMP PHY child node, not itself.
Move it to fix the dtbs_check warnings.

There are only '#clock-cells' removal from SM8350 QMP PHY nodes, because
child nodes already have the property.

Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Link: https://lore.kernel.org/r/20210929034253.24570-4-shawn.guo@linaro.org
arch/arm64/boot/dts/qcom/ipq8074.dtsi
arch/arm64/boot/dts/qcom/msm8996.dtsi
arch/arm64/boot/dts/qcom/msm8998.dtsi
arch/arm64/boot/dts/qcom/sm8350.dtsi

index 9ab4654..bf579a9 100644 (file)
@@ -91,7 +91,6 @@
                ssphy_1: phy@58000 {
                        compatible = "qcom,ipq8074-qmp-usb3-phy";
                        reg = <0x00058000 0x1c4>;
-                       #clock-cells = <1>;
                        #address-cells = <1>;
                        #size-cells = <1>;
                        ranges;
                                      <0x00058800 0x1f8>,     /* PCS  */
                                      <0x00058600 0x044>;     /* PCS misc*/
                                #phy-cells = <0>;
+                               #clock-cells = <1>;
                                clocks = <&gcc GCC_USB1_PIPE_CLK>;
                                clock-names = "pipe0";
                                clock-output-names = "gcc_usb1_pipe_clk_src";
                ssphy_0: phy@78000 {
                        compatible = "qcom,ipq8074-qmp-usb3-phy";
                        reg = <0x00078000 0x1c4>;
-                       #clock-cells = <1>;
                        #address-cells = <1>;
                        #size-cells = <1>;
                        ranges;
                                      <0x00078800 0x1f8>,     /* PCS  */
                                      <0x00078600 0x044>;     /* PCS misc*/
                                #phy-cells = <0>;
+                               #clock-cells = <1>;
                                clocks = <&gcc GCC_USB0_PIPE_CLK>;
                                clock-names = "pipe0";
                                clock-output-names = "gcc_usb0_pipe_clk_src";
index 465cd19..9b941b2 100644 (file)
                pcie_phy: phy@34000 {
                        compatible = "qcom,msm8996-qmp-pcie-phy";
                        reg = <0x00034000 0x488>;
-                       #clock-cells = <1>;
                        #address-cells = <1>;
                        #size-cells = <1>;
                        ranges;
                                      <0x00035400 0x1dc>;
                                #phy-cells = <0>;
 
+                               #clock-cells = <1>;
                                clock-output-names = "pcie_0_pipe_clk_src";
                                clocks = <&gcc GCC_PCIE_0_PIPE_CLK>;
                                clock-names = "pipe0";
                usb3phy: phy@7410000 {
                        compatible = "qcom,msm8996-qmp-usb3-phy";
                        reg = <0x07410000 0x1c4>;
-                       #clock-cells = <1>;
                        #address-cells = <1>;
                        #size-cells = <1>;
                        ranges;
                                      <0x07410600 0x1a8>;
                                #phy-cells = <0>;
 
+                               #clock-cells = <1>;
                                clock-output-names = "usb3_phy_pipe_clk_src";
                                clocks = <&gcc GCC_USB3_PHY_PIPE_CLK>;
                                clock-names = "pipe0";
index 887ba7c..789a3f6 100644 (file)
                        compatible = "qcom,msm8998-qmp-usb3-phy";
                        reg = <0x0c010000 0x18c>;
                        status = "disabled";
-                       #clock-cells = <1>;
                        #address-cells = <1>;
                        #size-cells = <1>;
                        ranges;
                                      <0xc010600 0x128>,
                                      <0xc010800 0x200>;
                                #phy-cells = <0>;
+                               #clock-cells = <1>;
                                clocks = <&gcc GCC_USB3_PHY_PIPE_CLK>;
                                clock-names = "pipe0";
                                clock-output-names = "usb3_phy_pipe_clk_src";
index b162518..45d5180 100644 (file)
                        reg = <0 0x01d87000 0 0xe10>;
                        #address-cells = <2>;
                        #size-cells = <2>;
-                       #clock-cells = <1>;
                        ranges;
                        clock-names = "ref",
                                      "ref_aux";
                              <0 0x088e8000 0 0x20>;
                        reg-names = "reg-base", "dp_com";
                        status = "disabled";
-                       #clock-cells = <1>;
                        #address-cells = <2>;
                        #size-cells = <2>;
                        ranges;
                        compatible = "qcom,sm8350-qmp-usb3-uni-phy";
                        reg = <0 0x088eb000 0 0x200>;
                        status = "disabled";
-                       #clock-cells = <1>;
                        #address-cells = <2>;
                        #size-cells = <2>;
                        ranges;