ARM: dts: imx25: Fix memory node duplication
authorFabio Estevam <festevam@gmail.com>
Mon, 26 Nov 2018 12:40:54 +0000 (10:40 -0200)
committerShawn Guo <shawnguo@kernel.org>
Mon, 10 Dec 2018 02:02:58 +0000 (10:02 +0800)
Boards based on imx25 have duplicate memory nodes:

- One coming from the board dts file: memory@

- One coming from the imx25.dtsi file.

Fix the duplication by removing the memory node from the dtsi file
and by adding 'device_type = "memory";' in the board dts.

Reported-by: Rob Herring <robh@kernel.org>
Signed-off-by: Fabio Estevam <festevam@gmail.com>
Signed-off-by: Shawn Guo <shawnguo@kernel.org>
arch/arm/boot/dts/imx25-eukrea-cpuimx25.dtsi
arch/arm/boot/dts/imx25-karo-tx25.dts
arch/arm/boot/dts/imx25-pdk.dts
arch/arm/boot/dts/imx25.dtsi

index e316fe0..e4d7da2 100644 (file)
@@ -18,6 +18,7 @@
        compatible = "eukrea,cpuimx25", "fsl,imx25";
 
        memory@80000000 {
+               device_type = "memory";
                reg = <0x80000000 0x4000000>; /* 64M */
        };
 };
index 5cb6967..f37e9a7 100644 (file)
@@ -37,6 +37,7 @@
        };
 
        memory@80000000 {
+               device_type = "memory";
                reg = <0x80000000 0x02000000 0x90000000 0x02000000>;
        };
 };
index a5626b4..f8544a9 100644 (file)
@@ -12,6 +12,7 @@
        compatible = "fsl,imx25-pdk", "fsl,imx25";
 
        memory@80000000 {
+               device_type = "memory";
                reg = <0x80000000 0x4000000>;
        };
 
index e801018..9a097ef 100644 (file)
         * The decompressor and also some bootloaders rely on a
         * pre-existing /chosen node to be available to insert the
         * command line and merge other ATAGS info.
-        * Also for U-Boot there must be a pre-existing /memory node.
         */
        chosen {};
-       memory { device_type = "memory"; };
 
        aliases {
                ethernet0 = &fec;