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

- One coming from the board dts file: memory@

- One coming from the imx27.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/imx27-apf27.dts
arch/arm/boot/dts/imx27-eukrea-cpuimx27.dtsi
arch/arm/boot/dts/imx27-pdk.dts
arch/arm/boot/dts/imx27-phytec-phycard-s-som.dtsi
arch/arm/boot/dts/imx27-phytec-phycore-som.dtsi
arch/arm/boot/dts/imx27.dtsi

index 3eddd80..f635d5c 100644 (file)
@@ -20,6 +20,7 @@
        compatible = "armadeus,imx27-apf27", "fsl,imx27";
 
        memory@a0000000 {
+               device_type = "memory";
                reg = <0xa0000000 0x04000000>;
        };
 };
index 9c455dc..c85f9d0 100644 (file)
@@ -17,6 +17,7 @@
        compatible = "eukrea,cpuimx27", "fsl,imx27";
 
        memory@a0000000 {
+               device_type = "memory";
                reg = <0xa0000000 0x04000000>;
        };
 
index f9a882d..35123b7 100644 (file)
@@ -10,6 +10,7 @@
        compatible = "fsl,imx27-pdk", "fsl,imx27";
 
        memory@a0000000 {
+               device_type = "memory";
                reg = <0xa0000000 0x08000000>;
        };
 
index cbad7c8..b0b4f7c 100644 (file)
@@ -18,6 +18,7 @@
        compatible = "phytec,imx27-pca100", "fsl,imx27";
 
        memory@a0000000 {
+               device_type = "memory";
                reg = <0xa0000000 0x08000000>; /* 128MB */
        };
 };
index ec466b4..0935e14 100644 (file)
@@ -17,6 +17,7 @@
        compatible = "phytec,imx27-pcm038", "fsl,imx27";
 
        memory@a0000000 {
+               device_type = "memory";
                reg = <0xa0000000 0x08000000>;
        };
 
index 151b0eb..26ff5d4 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;