Commit 6a968116 authored by Fabio Estevam's avatar Fabio Estevam Committed by Shawn Guo

ARM: dts: imx51: Fix memory node duplication

Boards based on imx51 have duplicate memory nodes:

- One coming from the board dts file: memory@

- One coming from the imx51.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: default avatarRob Herring <robh@kernel.org>
Signed-off-by: default avatarFabio Estevam <festevam@gmail.com>
Signed-off-by: default avatarShawn Guo <shawnguo@kernel.org>
parent 4951c2da
...@@ -22,6 +22,7 @@ / { ...@@ -22,6 +22,7 @@ / {
compatible = "armadeus,imx51-apf51", "fsl,imx51"; compatible = "armadeus,imx51-apf51", "fsl,imx51";
memory@90000000 { memory@90000000 {
device_type = "memory";
reg = <0x90000000 0x20000000>; reg = <0x90000000 0x20000000>;
}; };
......
...@@ -15,6 +15,7 @@ chosen { ...@@ -15,6 +15,7 @@ chosen {
}; };
memory@90000000 { memory@90000000 {
device_type = "memory";
reg = <0x90000000 0x20000000>; reg = <0x90000000 0x20000000>;
}; };
......
...@@ -17,6 +17,7 @@ / { ...@@ -17,6 +17,7 @@ / {
compatible = "digi,connectcore-ccxmx51-som", "fsl,imx51"; compatible = "digi,connectcore-ccxmx51-som", "fsl,imx51";
memory@90000000 { memory@90000000 {
device_type = "memory";
reg = <0x90000000 0x08000000>; reg = <0x90000000 0x08000000>;
}; };
}; };
......
...@@ -23,6 +23,7 @@ / { ...@@ -23,6 +23,7 @@ / {
compatible = "eukrea,cpuimx51", "fsl,imx51"; compatible = "eukrea,cpuimx51", "fsl,imx51";
memory@90000000 { memory@90000000 {
device_type = "memory";
reg = <0x90000000 0x10000000>; /* 256M */ reg = <0x90000000 0x10000000>; /* 256M */
}; };
}; };
......
...@@ -18,6 +18,7 @@ chosen { ...@@ -18,6 +18,7 @@ chosen {
}; };
memory@90000000 { memory@90000000 {
device_type = "memory";
reg = <0x90000000 0x10000000>; reg = <0x90000000 0x10000000>;
}; };
......
...@@ -53,6 +53,7 @@ chosen { ...@@ -53,6 +53,7 @@ chosen {
/* Will be filled by the bootloader */ /* Will be filled by the bootloader */
memory@90000000 { memory@90000000 {
device_type = "memory";
reg = <0x90000000 0>; reg = <0x90000000 0>;
}; };
......
...@@ -18,6 +18,7 @@ chosen { ...@@ -18,6 +18,7 @@ chosen {
/* Will be filled by the bootloader */ /* Will be filled by the bootloader */
memory@90000000 { memory@90000000 {
device_type = "memory";
reg = <0x90000000 0>; reg = <0x90000000 0>;
}; };
......
...@@ -18,6 +18,7 @@ chosen { ...@@ -18,6 +18,7 @@ chosen {
/* Will be filled by the bootloader */ /* Will be filled by the bootloader */
memory@90000000 { memory@90000000 {
device_type = "memory";
reg = <0x90000000 0>; reg = <0x90000000 0>;
}; };
......
...@@ -16,10 +16,8 @@ / { ...@@ -16,10 +16,8 @@ / {
* The decompressor and also some bootloaders rely on a * The decompressor and also some bootloaders rely on a
* pre-existing /chosen node to be available to insert the * pre-existing /chosen node to be available to insert the
* command line and merge other ATAGS info. * command line and merge other ATAGS info.
* Also for U-Boot there must be a pre-existing /memory node.
*/ */
chosen {}; chosen {};
memory { device_type = "memory"; };
aliases { aliases {
ethernet0 = &fec; ethernet0 = &fec;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment