ARM: dts: renesas: Fix USB PHY device and child node names
authorGeert Uytterhoeven <geert+renesas@glider.be>
Wed, 14 Sep 2022 14:21:54 +0000 (16:21 +0200)
committerGeert Uytterhoeven <geert+renesas@glider.be>
Mon, 19 Sep 2022 11:53:47 +0000 (13:53 +0200)
make dtbs_check:

    usb-phy@e6590100: '#phy-cells' is a required property
    From schema: dtschema/schemas/phy/phy-provider.yaml

The R-Car Gen2 USB PHY device nodes do not represent USB PHYs
theirselves, and thus do not have "#phy-cells" properties.  Fix the
warning by renaming them from "usb-phy" to "usb-phy-controller".
Rename their child nodes from "usb-channel" to "usb-phy", as these do
represent USB PHYs.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Link: https://lore.kernel.org/r/6442b4042e26537abc8632c4772f8201685f1f1f.1663165098.git.geert+renesas@glider.be
arch/arm/boot/dts/r8a7742.dtsi
arch/arm/boot/dts/r8a7743.dtsi
arch/arm/boot/dts/r8a7744.dtsi
arch/arm/boot/dts/r8a7745.dtsi
arch/arm/boot/dts/r8a77470.dtsi
arch/arm/boot/dts/r8a7790.dtsi
arch/arm/boot/dts/r8a7791.dtsi
arch/arm/boot/dts/r8a7794.dtsi

index 420e0b3..758a1bf 100644 (file)
                        status = "disabled";
                };
 
-               usbphy: usb-phy@e6590100 {
+               usbphy: usb-phy-controller@e6590100 {
                        compatible = "renesas,usb-phy-r8a7742",
                                     "renesas,rcar-gen2-usb-phy";
                        reg = <0 0xe6590100 0 0x100>;
                        resets = <&cpg 704>;
                        status = "disabled";
 
-                       usb0: usb-channel@0 {
+                       usb0: usb-phy@0 {
                                reg = <0>;
                                #phy-cells = <1>;
                        };
-                       usb2: usb-channel@2 {
+                       usb2: usb-phy@2 {
                                reg = <2>;
                                #phy-cells = <1>;
                        };
index c0c145a..20f1d98 100644 (file)
                        status = "disabled";
                };
 
-               usbphy: usb-phy@e6590100 {
+               usbphy: usb-phy-controller@e6590100 {
                        compatible = "renesas,usb-phy-r8a7743",
                                     "renesas,rcar-gen2-usb-phy";
                        reg = <0 0xe6590100 0 0x100>;
                        resets = <&cpg 704>;
                        status = "disabled";
 
-                       usb0: usb-channel@0 {
+                       usb0: usb-phy@0 {
                                reg = <0>;
                                #phy-cells = <1>;
                        };
-                       usb2: usb-channel@2 {
+                       usb2: usb-phy@2 {
                                reg = <2>;
                                #phy-cells = <1>;
                        };
index 3f4fb53..96b2d5a 100644 (file)
                        status = "disabled";
                };
 
-               usbphy: usb-phy@e6590100 {
+               usbphy: usb-phy-controller@e6590100 {
                        compatible = "renesas,usb-phy-r8a7744",
                                     "renesas,rcar-gen2-usb-phy";
                        reg = <0 0xe6590100 0 0x100>;
                        resets = <&cpg 704>;
                        status = "disabled";
 
-                       usb0: usb-channel@0 {
+                       usb0: usb-phy@0 {
                                reg = <0>;
                                #phy-cells = <1>;
                        };
-                       usb2: usb-channel@2 {
+                       usb2: usb-phy@2 {
                                reg = <2>;
                                #phy-cells = <1>;
                        };
index fe8e98a..afc902e 100644 (file)
                        status = "disabled";
                };
 
-               usbphy: usb-phy@e6590100 {
+               usbphy: usb-phy-controller@e6590100 {
                        compatible = "renesas,usb-phy-r8a7745",
                                     "renesas,rcar-gen2-usb-phy";
                        reg = <0 0xe6590100 0 0x100>;
                        resets = <&cpg 704>;
                        status = "disabled";
 
-                       usb0: usb-channel@0 {
+                       usb0: usb-phy@0 {
                                reg = <0>;
                                #phy-cells = <1>;
                        };
-                       usb2: usb-channel@2 {
+                       usb2: usb-phy@2 {
                                reg = <2>;
                                #phy-cells = <1>;
                        };
index c90f2a2..a5cf663 100644 (file)
                        status = "disabled";
                };
 
-               usbphy0: usb-phy@e6590100 {
+               usbphy0: usb-phy-controller@e6590100 {
                        compatible = "renesas,usb-phy-r8a77470",
                                     "renesas,rcar-gen2-usb-phy";
                        reg = <0 0xe6590100 0 0x100>;
                        resets = <&cpg 704>;
                        status = "disabled";
 
-                       usb0: usb-channel@0 {
+                       usb0: usb-phy@0 {
                                reg = <0>;
                                #phy-cells = <1>;
                        };
                        status = "disabled";
                };
 
-               usbphy1: usb-phy@e6598100 {
+               usbphy1: usb-phy-controller@e6598100 {
                        compatible = "renesas,usb-phy-r8a77470",
                                     "renesas,rcar-gen2-usb-phy";
                        reg = <0 0xe6598100 0 0x100>;
                        resets = <&cpg 706>;
                        status = "disabled";
 
-                       usb1: usb-channel@0 {
+                       usb1: usb-phy@0 {
                                reg = <0>;
                                #phy-cells = <1>;
                        };
index a640488..db171e3 100644 (file)
                        status = "disabled";
                };
 
-               usbphy: usb-phy@e6590100 {
+               usbphy: usb-phy-controller@e6590100 {
                        compatible = "renesas,usb-phy-r8a7790",
                                     "renesas,rcar-gen2-usb-phy";
                        reg = <0 0xe6590100 0 0x100>;
                        resets = <&cpg 704>;
                        status = "disabled";
 
-                       usb0: usb-channel@0 {
+                       usb0: usb-phy@0 {
                                reg = <0>;
                                #phy-cells = <1>;
                        };
-                       usb2: usb-channel@2 {
+                       usb2: usb-phy@2 {
                                reg = <2>;
                                #phy-cells = <1>;
                        };
index 542ed0a..d8f91d9 100644 (file)
                        status = "disabled";
                };
 
-               usbphy: usb-phy@e6590100 {
+               usbphy: usb-phy-controller@e6590100 {
                        compatible = "renesas,usb-phy-r8a7791",
                                     "renesas,rcar-gen2-usb-phy";
                        reg = <0 0xe6590100 0 0x100>;
                        resets = <&cpg 704>;
                        status = "disabled";
 
-                       usb0: usb-channel@0 {
+                       usb0: usb-phy@0 {
                                reg = <0>;
                                #phy-cells = <1>;
                        };
-                       usb2: usb-channel@2 {
+                       usb2: usb-phy@2 {
                                reg = <2>;
                                #phy-cells = <1>;
                        };
index b601ee6..7aa781f 100644 (file)
                        status = "disabled";
                };
 
-               usbphy: usb-phy@e6590100 {
+               usbphy: usb-phy-controller@e6590100 {
                        compatible = "renesas,usb-phy-r8a7794",
                                     "renesas,rcar-gen2-usb-phy";
                        reg = <0 0xe6590100 0 0x100>;
                        resets = <&cpg 704>;
                        status = "disabled";
 
-                       usb0: usb-channel@0 {
+                       usb0: usb-phy@0 {
                                reg = <0>;
                                #phy-cells = <1>;
                        };
-                       usb2: usb-channel@2 {
+                       usb2: usb-phy@2 {
                                reg = <2>;
                                #phy-cells = <1>;
                        };