Commit e021f0cc authored by Fabio Estevam's avatar Fabio Estevam Committed by Greg Kroah-Hartman

ARM: dts: imx7: Fix memory node duplication

[ Upstream commit 29988e86 ]

Boards based on imx7 have duplicate memory nodes:

- One coming from the board dts file: memory@

- One coming from the imx7s.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>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent a9046934
...@@ -19,6 +19,7 @@ / { ...@@ -19,6 +19,7 @@ / {
compatible = "compulab,cl-som-imx7", "fsl,imx7d"; compatible = "compulab,cl-som-imx7", "fsl,imx7d";
memory@80000000 { memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x10000000>; /* 256 MB - minimal configuration */ reg = <0x80000000 0x10000000>; /* 256 MB - minimal configuration */
}; };
......
...@@ -8,6 +8,7 @@ ...@@ -8,6 +8,7 @@
/ { / {
memory@80000000 { memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x40000000>; reg = <0x80000000 0x40000000>;
}; };
}; };
......
...@@ -45,6 +45,7 @@ ...@@ -45,6 +45,7 @@
/ { / {
memory@80000000 { memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x20000000>; reg = <0x80000000 0x20000000>;
}; };
}; };
......
...@@ -12,6 +12,7 @@ / { ...@@ -12,6 +12,7 @@ / {
compatible = "boundary,imx7d-nitrogen7", "fsl,imx7d"; compatible = "boundary,imx7d-nitrogen7", "fsl,imx7d";
memory@80000000 { memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x40000000>; reg = <0x80000000 0x40000000>;
}; };
......
...@@ -49,6 +49,7 @@ / { ...@@ -49,6 +49,7 @@ / {
compatible = "technexion,imx7d-pico", "fsl,imx7d"; compatible = "technexion,imx7d-pico", "fsl,imx7d";
memory@80000000 { memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x80000000>; reg = <0x80000000 0x80000000>;
}; };
......
...@@ -15,6 +15,7 @@ chosen { ...@@ -15,6 +15,7 @@ chosen {
}; };
memory@80000000 { memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x80000000>; reg = <0x80000000 0x80000000>;
}; };
......
...@@ -45,6 +45,7 @@ ...@@ -45,6 +45,7 @@
/ { / {
memory@80000000 { memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x10000000>; reg = <0x80000000 0x10000000>;
}; };
}; };
...@@ -51,6 +51,7 @@ / { ...@@ -51,6 +51,7 @@ / {
compatible = "warp,imx7s-warp", "fsl,imx7s"; compatible = "warp,imx7s-warp", "fsl,imx7s";
memory@80000000 { memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x20000000>; reg = <0x80000000 0x20000000>;
}; };
......
...@@ -17,10 +17,8 @@ / { ...@@ -17,10 +17,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 {
gpio0 = &gpio1; gpio0 = &gpio1;
......
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