OSDN Git Service

ARM: dts: armada-370: Fixup pcie DT warnings
authorGregory CLEMENT <gregory.clement@free-electrons.com>
Sat, 5 Nov 2016 18:20:09 +0000 (19:20 +0100)
committerGregory CLEMENT <gregory.clement@free-electrons.com>
Sat, 19 Nov 2016 08:16:38 +0000 (09:16 +0100)
PCIe has a ranges property, so the unit name should contain an address.
Take the opportunity to use the node label instead of the full name.

Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
arch/arm/boot/dts/armada-370-db.dts
arch/arm/boot/dts/armada-370-dlink-dns327l.dts
arch/arm/boot/dts/armada-370-mirabox.dts
arch/arm/boot/dts/armada-370-netgear-rn102.dts
arch/arm/boot/dts/armada-370-netgear-rn104.dts
arch/arm/boot/dts/armada-370-rd.dts
arch/arm/boot/dts/armada-370-seagate-nas-4bay.dts
arch/arm/boot/dts/armada-370-seagate-nas-xbay.dtsi
arch/arm/boot/dts/armada-370-seagate-personal-cloud.dtsi
arch/arm/boot/dts/armada-370.dtsi

index d261150..f3adc18 100644 (file)
                                };
                        };
                };
-
-               pcie-controller {
-                       status = "okay";
-                       /*
-                        * The two PCIe units are accessible through
-                        * both standard PCIe slots and mini-PCIe
-                        * slots on the board.
-                        */
-                       pcie@1,0 {
-                               /* Port 0, Lane 0 */
-                               status = "okay";
-                       };
-
-                       pcie@2,0 {
-                               /* Port 1, Lane 0 */
-                               status = "okay";
-                       };
-               };
        };
 
        sound {
                compatible = "linux,spdif-dir";
        };
 };
+
+&pciec {
+       status = "okay";
+       /*
+        * The two PCIe units are accessible through
+        * both standard PCIe slots and mini-PCIe
+        * slots on the board.
+        */
+       pcie@1,0 {
+               /* Port 0, Lane 0 */
+               status = "okay";
+       };
+
+       pcie@2,0 {
+               /* Port 1, Lane 0 */
+               status = "okay";
+       };
+};
+
 &mdio {
        pinctrl-0 = <&mdio_pins>;
        pinctrl-names = "default";
index e2a363b..9e499ee 100644 (file)
                        MBUS_ID(0x01, 0xe0) 0 0xfff00000 0x100000
                        MBUS_ID(0x09, 0x01) 0 0xf1100000 0x10000>;
 
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               /* Port 0, Lane 0 */
-                               status = "okay";
-                       };
-
-                       pcie@2,0 {
-                               /* Port 1, Lane 0 */
-                               status = "okay";
-                       };
-               };
-
                internal-regs {
                        sata@a0000 {
                                nr-ports = <2>;
        };
 };
 
+&pciec {
+       status = "okay";
+
+       pcie@1,0 {
+               /* Port 0, Lane 0 */
+               status = "okay";
+       };
+
+       pcie@2,0 {
+               /* Port 1, Lane 0 */
+               status = "okay";
+       };
+};
+
 &pinctrl {
        sata_l_white_pin: sata-l-white-pin {
                marvell,pins = "mpp57";
index 3e1ef56..c4dded0 100644 (file)
                          MBUS_ID(0x01, 0xe0) 0 0xfff00000 0x100000
                          MBUS_ID(0x09, 0x01) 0 0xf1100000 0x10000>;
 
-               pcie-controller {
-                       status = "okay";
-
-                       /* Internal mini-PCIe connector */
-                       pcie@1,0 {
-                               /* Port 0, Lane 0 */
-                               status = "okay";
-                       };
-
-                       /* Connected on the PCB to a USB 3.0 XHCI controller */
-                       pcie@2,0 {
-                               /* Port 1, Lane 0 */
-                               status = "okay";
-                       };
-               };
-
                internal-regs {
                        serial@12000 {
                                status = "okay";
        };
 };
 
+&pciec {
+       status = "okay";
+
+       /* Internal mini-PCIe connector */
+       pcie@1,0 {
+               /* Port 0, Lane 0 */
+               status = "okay";
+       };
+
+       /* Connected on the PCB to a USB 3.0 XHCI controller */
+       pcie@2,0 {
+               /* Port 1, Lane 0 */
+               status = "okay";
+       };
+};
+
 &mdio {
        pinctrl-0 = <&mdio_pins>;
        pinctrl-names = "default";
index a640985..99b9d73 100644 (file)
                          MBUS_ID(0x01, 0xe0) 0 0xfff00000 0x100000
                          MBUS_ID(0x09, 0x01) 0 0xf1100000 0x10000>;
 
-               pcie-controller {
-                       status = "okay";
-
-                       /* Connected to Marvell 88SE9170 SATA controller */
-                       pcie@1,0 {
-                               /* Port 0, Lane 0 */
-                               status = "okay";
-                       };
-
-                       /* Connected to FL1009 USB 3.0 controller */
-                       pcie@2,0 {
-                               /* Port 1, Lane 0 */
-                               status = "okay";
-                       };
-               };
-
                internal-regs {
 
                        /* RTC is provided by Intersil ISL12057 I2C RTC chip */
        };
 };
 
+&pciec {
+       status = "okay";
+
+       /* Connected to Marvell 88SE9170 SATA controller */
+       pcie@1,0 {
+               /* Port 0, Lane 0 */
+               status = "okay";
+       };
+
+       /* Connected to FL1009 USB 3.0 controller */
+       pcie@2,0 {
+               /* Port 1, Lane 0 */
+               status = "okay";
+       };
+};
+
 &mdio {
        pinctrl-0 = <&mdio_pins>;
        pinctrl-names = "default";
index fd5f1d9..b5dd301 100644 (file)
                          MBUS_ID(0x01, 0xe0) 0 0xfff00000 0x100000
                          MBUS_ID(0x09, 0x01) 0 0xf1100000 0x10000>;
 
-               pcie-controller {
-                       status = "okay";
-
-                       /* Connected to FL1009 USB 3.0 controller */
-                       pcie@1,0 {
-                               /* Port 0, Lane 0 */
-                               status = "okay";
-                       };
-
-                       /* Connected to Marvell 88SE9215 SATA controller */
-                       pcie@2,0 {
-                               /* Port 1, Lane 0 */
-                               status = "okay";
-                       };
-               };
-
                internal-regs {
 
                        /* RTC is provided by Intersil ISL12057 I2C RTC chip */
        };
 };
 
+&pciec {
+       status = "okay";
+
+       /* Connected to FL1009 USB 3.0 controller */
+       pcie@1,0 {
+               /* Port 0, Lane 0 */
+               status = "okay";
+       };
+
+       /* Connected to Marvell 88SE9215 SATA controller */
+       pcie@2,0 {
+               /* Port 1, Lane 0 */
+               status = "okay";
+       };
+};
+
 &mdio {
        pinctrl-0 = <&mdio_pins>;
        pinctrl-names = "default";
index 621add7..29722b9 100644 (file)
                          MBUS_ID(0x01, 0xe0) 0 0xfff00000 0x100000
                          MBUS_ID(0x09, 0x01) 0 0xf1100000 0x10000>;
 
-               pcie-controller {
-                       status = "okay";
-
-                       /* Internal mini-PCIe connector */
-                       pcie@1,0 {
-                               /* Port 0, Lane 0 */
-                               status = "okay";
-                       };
-
-                       /* Internal mini-PCIe connector */
-                       pcie@2,0 {
-                               /* Port 1, Lane 0 */
-                               status = "okay";
-                       };
-               };
-
                internal-regs {
                        serial@12000 {
                                status = "okay";
         };
 };
 
+&pciec {
+       status = "okay";
+
+       /* Internal mini-PCIe connector */
+       pcie@1,0 {
+               /* Port 0, Lane 0 */
+               status = "okay";
+       };
+
+       /* Internal mini-PCIe connector */
+       pcie@2,0 {
+               /* Port 1, Lane 0 */
+               status = "okay";
+       };
+};
+
 &mdio {
        pinctrl-0 = <&mdio_pins>;
        pinctrl-names = "default";
index 82ce5ec..eb6af53 100644 (file)
        compatible = "seagate,dart-4", "marvell,armada370", "marvell,armada-370-xp";
 
        soc {
-               pcie-controller {
-                       /* SATA AHCI controller 88SE9170 */
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-
                internal-regs {
                        ethernet@74000 {
                                status = "okay";
        };
 };
 
+&pciec {
+       /* SATA AHCI controller 88SE9170 */
+       pcie@1,0 {
+               status = "okay";
+       };
+};
+
 &mdio {
        phy1: ethernet-phy@1 {
                reg = <1>;
index 724a479..9b582c4 100644 (file)
                ranges = <MBUS_ID(0xf0, 0x01) 0 0xf1000000 0x100000
                          MBUS_ID(0x01, 0xe0) 0 0xfff00000 0x100000>;
 
-               pcie-controller {
-                       status = "okay";
-
-                       /* USB 3.0 bridge ASM1042A */
-                       pcie@2,0 {
-                               status = "okay";
-                       };
-               };
-
                internal-regs {
                        serial@12000 {
                                status = "okay";
        };
 };
 
+&pciec {
+       status = "okay";
+
+       /* USB 3.0 bridge ASM1042A */
+       pcie@2,0 {
+               status = "okay";
+       };
+};
+
+
 &mdio {
        pinctrl-0 = <&mdio_pins>;
        pinctrl-names = "default";
index fb52a34..bb678a4 100644 (file)
                ranges = <MBUS_ID(0xf0, 0x01) 0 0xf1000000 0x100000
                          MBUS_ID(0x01, 0xe0) 0 0xfff00000 0x100000>;
 
-               pcie-controller {
-                       status = "okay";
-
-                       /* USB 3.0 Bridge ASM1042A */
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-
                internal-regs {
                        coherency-fabric@20200 {
                                broken-idle;
        };
 };
 
+&pciec {
+       status = "okay";
+
+       /* USB 3.0 Bridge ASM1042A */
+       pcie@1,0 {
+               status = "okay";
+       };
+};
+
 &mdio {
        pinctrl-0 = <&mdio_pins>;
        pinctrl-names = "default";
index 079494e..60cdeb3 100644 (file)
@@ -70,7 +70,7 @@
                        reg = <MBUS_ID(0x01, 0xe0) 0 0x100000>;
                };
 
-               pciec: pcie-controller {
+               pciec: pcie-controller@82000000 {
                        compatible = "marvell,armada-370-pcie";
                        status = "disabled";
                        device_type = "pci";