ARM: dts: imx35: Fix memory node duplication

Boards based on imx35 have duplicate memory nodes:

- One coming from the board dts file: memory@

- One coming from the imx35.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>

diff --git a/arch/arm/boot/dts/imx35-eukrea-cpuimx35.dtsi b/arch/arm/boot/dts/imx35-eukrea-cpuimx35.dtsi
index ba39d938f2891a3485e950cc3491578910a1b306..5f8a47a9fcd4071f7ce697111d6e4170ab9c1b19 100644 (file)
--- a/arch/arm/boot/dts/imx35-eukrea-cpuimx35.dtsi
+++ b/arch/arm/boot/dts/imx35-eukrea-cpuimx35.dtsi
@@ -18,6 +18,7 @@ / {
        compatible = "eukrea,cpuimx35", "fsl,imx35";
 
        memory@80000000 {
+               device_type = "memory";
                reg = <0x80000000 0x8000000>; /* 128M */
        };
 };

diff --git a/arch/arm/boot/dts/imx35-pdk.dts b/arch/arm/boot/dts/imx35-pdk.dts
index df613e88fd2c19622aee24020d215c4add0341e5..ddce0a844758b3a5836c3d240f0f13bff6a68f4a 100644 (file)
--- a/arch/arm/boot/dts/imx35-pdk.dts
+++ b/arch/arm/boot/dts/imx35-pdk.dts
@@ -11,6 +11,7 @@ / {
        compatible = "fsl,imx35-pdk", "fsl,imx35";
 
        memory@80000000 {
+               device_type = "memory";
                reg = <0x80000000 0x8000000>,
                      <0x90000000 0x8000000>;
        };

diff --git a/arch/arm/boot/dts/imx35.dtsi b/arch/arm/boot/dts/imx35.dtsi
index a1c3d28e8771c23eaa3b385e3ef901958ee57adb..59cadeee23eddcfe1271654539b9375326d652d0 100644 (file)
--- a/arch/arm/boot/dts/imx35.dtsi
+++ b/arch/arm/boot/dts/imx35.dtsi
@@ -13,10 +13,8 @@ / {
         * 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;