ARM: dts: kirkwood: Fixup pcie DT warnings
authorAndrew Lunn <andrew@lunn.ch>
Sun, 3 Apr 2016 02:03:47 +0000 (04:03 +0200)
committerGregory CLEMENT <gregory.clement@free-electrons.com>
Thu, 14 Apr 2016 17:06:15 +0000 (19:06 +0200)
PCIe has a range property, so the unit name should contain an address.
Make use of the label to enable individual PCIe busses. Also, fixup
the synology dtsi file which added a label pcie2 rather than using the
existing pcie1 label.

Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
35 files changed:
arch/arm/boot/dts/kirkwood-6192.dtsi
arch/arm/boot/dts/kirkwood-6281.dtsi
arch/arm/boot/dts/kirkwood-6282.dtsi
arch/arm/boot/dts/kirkwood-98dx4122.dtsi
arch/arm/boot/dts/kirkwood-b3.dts
arch/arm/boot/dts/kirkwood-db-88f6281.dts
arch/arm/boot/dts/kirkwood-db-88f6282.dts
arch/arm/boot/dts/kirkwood-dir665.dts
arch/arm/boot/dts/kirkwood-ds111.dts
arch/arm/boot/dts/kirkwood-ds112.dts
arch/arm/boot/dts/kirkwood-ds212.dts
arch/arm/boot/dts/kirkwood-ds411.dts
arch/arm/boot/dts/kirkwood-ds411slim.dts
arch/arm/boot/dts/kirkwood-iconnect.dts
arch/arm/boot/dts/kirkwood-km_common.dtsi
arch/arm/boot/dts/kirkwood-laplug.dts
arch/arm/boot/dts/kirkwood-linkstation.dtsi
arch/arm/boot/dts/kirkwood-mplcec4.dts
arch/arm/boot/dts/kirkwood-mv88f6281gtw-ge.dts
arch/arm/boot/dts/kirkwood-nas2big.dts
arch/arm/boot/dts/kirkwood-netgear_readynas_duo_v2.dts
arch/arm/boot/dts/kirkwood-netgear_readynas_nv+_v2.dts
arch/arm/boot/dts/kirkwood-nsa310.dts
arch/arm/boot/dts/kirkwood-nsa320.dts
arch/arm/boot/dts/kirkwood-nsa325.dts
arch/arm/boot/dts/kirkwood-nsa3x0-common.dtsi
arch/arm/boot/dts/kirkwood-openrd.dtsi
arch/arm/boot/dts/kirkwood-rd88f6192.dts
arch/arm/boot/dts/kirkwood-rd88f6281.dtsi
arch/arm/boot/dts/kirkwood-rs212.dts
arch/arm/boot/dts/kirkwood-synology.dtsi
arch/arm/boot/dts/kirkwood-t5325.dts
arch/arm/boot/dts/kirkwood-ts219-6282.dts
arch/arm/boot/dts/kirkwood-ts219.dtsi
arch/arm/boot/dts/kirkwood-ts419-6282.dts

index 9e6e9e2..1014a5d 100644 (file)
@@ -1,6 +1,6 @@
 / {
        mbus {
-               pciec: pcie-controller {
+               pciec: pcie-controller@82000000 {
                        compatible = "marvell,kirkwood-pcie";
                        status = "disabled";
                        device_type = "pci";
index 7dc7d67..fad2015 100644 (file)
@@ -1,6 +1,6 @@
 / {
        mbus {
-               pciec: pcie-controller {
+               pciec: pcie-controller@82000000 {
                        compatible = "marvell,kirkwood-pcie";
                        status = "disabled";
                        device_type = "pci";
index 4680eec..27434c3 100644 (file)
@@ -1,6 +1,6 @@
 / {
        mbus {
-               pciec: pcie-controller {
+               pciec: pcie-controller@82000000 {
                        compatible = "marvell,kirkwood-pcie";
                        status = "disabled";
                        device_type = "pci";
index 9e1f741..17eb927 100644 (file)
@@ -1,6 +1,6 @@
 / {
        mbus {
-               pciec: pcie-controller {
+               pciec: pcie-controller@82000000 {
                        compatible = "marvell,kirkwood-pcie";
                        status = "disabled";
                        device_type = "pci";
index d2936ad..d091ecb 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       /* Wifi model has Atheros chipset on pcie port */
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
                        pmx_button_power: pmx-button-power {
        };
 };
 
+/* Wifi model has Atheros chipset on pcie port */
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index c39dd76..aee6f02 100644 (file)
 / {
        model = "Marvell DB-88F6281-BP Development Board";
        compatible = "marvell,db-88f6281-bp", "marvell,kirkwood-88f6281", "marvell,kirkwood";
+};
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
+&pciec {
+        status = "okay";
+};
 
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
+&pcie0 {
+       status = "okay";
 };
index 701c6b6..e8b23e1 100644 (file)
 / {
        model = "Marvell DB-88F6282-BP Development Board";
        compatible = "marvell,db-88f6282-bp", "marvell,kirkwood-88f6282", "marvell,kirkwood";
+};
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
+&pciec {
+        status = "okay";
+};
 
-                       pcie@1,0 {
-                               status = "okay";
-                       };
+&pcie0 {
+       status = "okay";
+};
 
-                       pcie@2,0 {
-                               status = "okay";
-                       };
-               };
-       };
+&pcie1 {
+       status = "okay";
 };
index f548363..41acbb6 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
                        pinctrl-0 =< &pmx_led_usb
 &rtc {
        status = "disabled";
 };
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index 61f47fb..a85a466 100644 (file)
@@ -40,6 +40,6 @@
        status = "okay";
 };
 
-&pcie2 {
+&pcie1 {
        status = "okay";
 };
index b84af3d..6cef4bd 100644 (file)
        status = "okay";
 };
 
-&pcie2 {
+&pciec {
+        status = "okay";
+};
+
+&pcie1 {
        status = "okay";
 };
index 99afd46..7f32e7a 100644 (file)
@@ -43,6 +43,6 @@
        status = "okay";
 };
 
-&pcie2 {
+&pcie1 {
        status = "okay";
 };
index 623cd4a..72e5830 100644 (file)
        status = "okay";
 };
 
-&pcie2 {
+&pciec {
+        status = "okay";
+};
+
+&pcie1 {
        status = "okay";
 };
index a0a1fad..aaaf31b 100644 (file)
@@ -44,6 +44,6 @@
        status = "okay";
 };
 
-&pcie2 {
+&pcie1 {
        status = "okay";
 };
index 6743067..d25184a 100644 (file)
                linux,initrd-end   = <0x4800000>;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
                        pmx_button_reset: pmx-button-reset {
                phy-handle = <&ethphy0>;
        };
 };
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index 8367c77..f6096f5 100644 (file)
@@ -4,16 +4,6 @@
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
                        pinctrl-0 = < &pmx_i2c_gpio_sda &pmx_i2c_gpio_scl >;
        status = "okay";
        chip-delay = <25>;
 };
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index c39e17a..1b0f070 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                serial@12000 {
                        status = "okay";
                phy-handle = <&ethphy0>;
        };
 };
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index 69061b6..36c54c9 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
                        pmx_power_hdd0: pmx-power-hdd0 {
                };
        };
 };
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index f3a9918..aa413b0 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
                        pmx_led_health: pmx-led-health {
                phy-handle = <&ethphy1>;
        };
 };
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index 26fb610..172a38c 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-        };
-
        ocp@f1000000 {
                pin-controller@10000 {
                        pmx_usb_led: pmx-usb-led {
                duplex = <1>;
        };
 };
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index 7427ec5..f53bcac 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                rtc@10300 {
                        /* The on-chip RTC is not powered (no supercap). */
                reg = <0x9100000 0x6f00000>;
        };
 };
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index fd733c6..8745484 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
                        pmx_button_power: pmx-button-power {
                phy-handle = <&ethphy0>;
        };
 };
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index b514d64..a4e97f7 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       /* Connected to NEC uPD720200 USB 3.0 controller */
-                       pcie@1,0 {
-                               /* Port 0, Lane 0 */
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
                        pmx_button_power: pmx-button-power {
                phy-handle = <&ethphy0>;
        };
 };
+
+/* Connected to NEC uPD720200 USB 3.0 controller */
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index 6139df0..0b69ee4 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
                        pinctrl-0 = <&pmx_unknown>;
                };
        };
 };
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index 2a935e9..6ab104b 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
                        pinctrl-names = "default";
                phy-handle = <&ethphy0>;
        };
 };
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index bc4ec93..36c6481 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
                        pinctrl-names = "default";
        };
 };
 
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index 5ccc461..e09b79a 100644 (file)
@@ -4,16 +4,6 @@
 / {
        model = "ZyXEL NSA310";
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
 
                reg = <0x5040000 0x2fc0000>;
        };
 };
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index 24f1d30..e4ecab1 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
                        pinctrl-0 = <&pmx_select28 &pmx_sdio_cd &pmx_select34>;
                reg = <0x0600000 0x1FA00000>;
        };
 };
+
+&pcie0 {
+       status = "okay";
+};
index e0b9593..b8af907 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
                        pinctrl-0 = <&pmx_usb_power>;
         ethernet0-port@0 {
                 phy-handle = <&ethphy0>;
         };
-};
\ No newline at end of file
+};
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index 5521426..d5aacf1 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
                        pinctrl-names = "default";
                duplex = <1>;
        };
 };
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index 3b19f1f..2c722ec 100644 (file)
        status = "okay";
 };
 
-&pcie2 {
+&pciec {
+        status = "okay";
+};
+
+&pcie1 {
        status = "okay";
 };
index 04015c1..65e9524 100644 (file)
  */
 
 / {
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-
-                       pcie2: pcie@2,0 {
-                               status = "disabled";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
                        pmx_alarmled_12: pmx-alarmled-12 {
                phy-handle = <&ethphy1>;
        };
 };
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index 860c6d7..3500f47 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
                        pinctrl-0 = <&pmx_i2s &pmx_sysrst>;
                phy-handle = <&ethphy0>;
        };
 };
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index 1d291f6..3437bb3 100644 (file)
@@ -5,16 +5,6 @@
 #include "kirkwood-ts219.dtsi"
 
 / {
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@2,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                pinctrl: pin-controller@10000 {
 
@@ -63,3 +53,5 @@
 };
 
 &ethphy0 { reg = <0>; };
+
+&pcie1 { status = "okay"; };
index b3f7538..62e5e2d 100644 (file)
                stdout-path = &uart0;
        };
 
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@1,0 {
-                               status = "okay";
-                       };
-               };
-       };
-
        ocp@f1000000 {
                i2c@11000 {
                        status = "okay";
                phy-handle = <&ethphy0>;
        };
 };
+
+&pciec {
+        status = "okay";
+};
+
+&pcie0 {
+       status = "okay";
+};
index d7512d4..e3e71f4 100644 (file)
 #include "kirkwood-ts219.dtsi"
 #include "kirkwood-ts419.dtsi"
 
-/ {
-       mbus {
-               pcie-controller {
-                       status = "okay";
-
-                       pcie@2,0 {
-                               status = "okay";
-                       };
-               };
-       };
-};
-
 &ethphy0 { reg = <0>; };
 &ethphy1 { reg = <1>; };
+
+&pciec { status = "okay"; };
+&pcie1 { status = "okay"; };