Instead of duplicating the node hierarchy, reference the nodes by label,
adding labels where necessary.

Drop some trailing or inconsistent white lines while at it.

Fixes: ec7e5a569bce ("arm64: dts: marvell: add Device Tree files for Armada 
7K/8K")
Cc: Thomas Petazzoni <[email protected]>
Signed-off-by: Andreas Färber <[email protected]>
---
 Is it intentional that there are no aliases for the two serial nodes and no 
/chosen/stdout-path?
 
 arch/arm64/boot/dts/marvell/armada-7040-db.dts     | 52 ++++++++++------------
 arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi |  1 -
 arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi |  2 -
 arch/arm64/boot/dts/marvell/armada-ap806.dtsi      |  7 +--
 4 files changed, 26 insertions(+), 36 deletions(-)

diff --git a/arch/arm64/boot/dts/marvell/armada-7040-db.dts 
b/arch/arm64/boot/dts/marvell/armada-7040-db.dts
index 064a251346dd..ee5778395bea 100644
--- a/arch/arm64/boot/dts/marvell/armada-7040-db.dts
+++ b/arch/arm64/boot/dts/marvell/armada-7040-db.dts
@@ -55,38 +55,34 @@
                device_type = "memory";
                reg = <0x0 0x0 0x0 0x80000000>;
        };
+};
 
-       ap806 {
-               config-space {
-                       spi@510600 {
-                               status = "okay";
-
-                               spi-flash@0 {
-                                       #address-cells = <1>;
-                                       #size-cells = <1>;
-                                       compatible = "n25q128a13";
-                                       reg = <0>; /* Chip select 0 */
-                                       spi-max-frequency = <10000000>;
+&i2c0 {
+       status = "okay";
+       clock-frequency = <100000>;
+};
 
-                                       partition@0 {
-                                               label = "U-Boot";
-                                               reg = <0 0x200000>;
-                                       };
-                                       partition@400000 {
-                                               label = "Filesystem";
-                                               reg = <0x200000 0xce0000>;
-                                       };
-                               };
-                       };
+&spi0 {
+       status = "okay";
 
-                       i2c@511000 {
-                               status = "okay";
-                               clock-frequency = <100000>;
-                       };
+       spi-flash@0 {
+               #address-cells = <1>;
+               #size-cells = <1>;
+               compatible = "n25q128a13";
+               reg = <0>; /* Chip select 0 */
+               spi-max-frequency = <10000000>;
 
-                       serial@512000 {
-                               status = "okay";
-                       };
+               partition@0 {
+                       label = "U-Boot";
+                       reg = <0 0x200000>;
+               };
+               partition@400000 {
+                       label = "Filesystem";
+                       reg = <0x200000 0xce0000>;
                };
        };
 };
+
+&uart0 {
+       status = "okay";
+};
diff --git a/arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi 
b/arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi
index f25c5c17fad7..95a1ff60f6c1 100644
--- a/arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-ap806-dual.dtsi
@@ -68,4 +68,3 @@
                };
        };
 };
-
diff --git a/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi 
b/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi
index baa7d9a516b3..ba43a4357b89 100644
--- a/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-ap806-quad.dtsi
@@ -79,6 +79,4 @@
                        enable-method = "psci";
                };
        };
-
 };
-
diff --git a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi 
b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi
index 556a92bcc2f6..3ecf9b1798fa 100644
--- a/arch/arm64/boot/dts/marvell/armada-ap806.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-ap806.dtsi
@@ -59,7 +59,6 @@
                method = "smc";
        };
 
-
        ap806 {
                #address-cells = <2>;
                #size-cells = <2>;
@@ -190,7 +189,7 @@
                                status = "disabled";
                        };
 
-                       serial@512000 {
+                       uart0: serial@512000 {
                                compatible = "snps,dw-apb-uart";
                                reg = <0x512000 0x100>;
                                reg-shift = <2>;
@@ -200,7 +199,7 @@
                                status = "disabled";
                        };
 
-                       serial@512100 {
+                       uart1: serial@512100 {
                                compatible = "snps,dw-apb-uart";
                                reg = <0x512100 0x100>;
                                reg-shift = <2>;
@@ -232,6 +231,4 @@
                        };
                };
        };
-
 };
-
-- 
2.6.2

Reply via email to