Commit 23866a3d authored by Masahiro Yamada's avatar Masahiro Yamada

ARM: dts: uniphier: move memory node below aliases node

These UniPhier DT files are fine as long as they are compiled in the
Linux build system.  It is true that Linux is the biggest user of
DT, but DT is project neutral from its concept.  DT files are often
re-used for other projects.  Especially for the UniPhier platform,
these DT files are re-used for U-Boot as well.

If I feed these DT files to the FDTGREP tool in U-Boot, it complains
about the node order.

  FDTGREP spl/u-boot-spl.dtb
  Error at 'fdt_find_regions': FDT_ERR_BADLAYOUT
  /aliases node must come before all other nodes

Given that DT is not very sensitive to the order of nodes, this is a
problem of FDTGREP.  I filed a bug report a year ago, but it has not
been fixed yet.

Differentiating DT is painful.  So, I am up-streaming the requirement
from the down-stream project.
Signed-off-by: default avatarMasahiro Yamada <yamada.masahiro@socionext.com>
parent 2201c7f1
......@@ -52,11 +52,6 @@ / {
model = "UniPhier LD4 Reference Board";
compatible = "socionext,uniphier-ld4-ref", "socionext,uniphier-ld4";
memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x20000000>;
};
chosen {
stdout-path = "serial0:115200n8";
};
......@@ -71,6 +66,11 @@ aliases {
i2c2 = &i2c2;
i2c3 = &i2c3;
};
memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x20000000>;
};
};
&ethsc {
......
......@@ -52,11 +52,6 @@ / {
model = "UniPhier LD6b Reference Board";
compatible = "socionext,uniphier-ld6b-ref", "socionext,uniphier-ld6b";
memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x80000000>;
};
chosen {
stdout-path = "serial0:115200n8";
};
......@@ -73,6 +68,11 @@ aliases {
i2c5 = &i2c5;
i2c6 = &i2c6;
};
memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x80000000>;
};
};
&ethsc {
......
......@@ -50,11 +50,6 @@ / {
model = "UniPhier Pro4 Ace Board";
compatible = "socionext,uniphier-pro4-ace", "socionext,uniphier-pro4";
memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x40000000>;
};
chosen {
stdout-path = "serial0:115200n8";
};
......@@ -70,6 +65,11 @@ aliases {
i2c5 = &i2c5;
i2c6 = &i2c6;
};
memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x40000000>;
};
};
&serial0 {
......
......@@ -52,11 +52,6 @@ / {
model = "UniPhier Pro4 Reference Board";
compatible = "socionext,uniphier-pro4-ref", "socionext,uniphier-pro4";
memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x40000000>;
};
chosen {
stdout-path = "serial0:115200n8";
};
......@@ -73,6 +68,11 @@ aliases {
i2c5 = &i2c5;
i2c6 = &i2c6;
};
memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x40000000>;
};
};
&ethsc {
......
......@@ -50,11 +50,6 @@ / {
model = "UniPhier Pro4 Sanji Board";
compatible = "socionext,uniphier-pro4-sanji", "socionext,uniphier-pro4";
memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x80000000>;
};
chosen {
stdout-path = "serial0:115200n8";
};
......@@ -69,6 +64,11 @@ aliases {
i2c5 = &i2c5;
i2c6 = &i2c6;
};
memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x80000000>;
};
};
&serial0 {
......
......@@ -51,11 +51,6 @@ / {
compatible = "socionext,uniphier-pxs2-gentil",
"socionext,uniphier-pxs2";
memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x80000000>;
};
chosen {
stdout-path = "serial0:115200n8";
};
......@@ -70,6 +65,11 @@ aliases {
i2c5 = &i2c5;
i2c6 = &i2c6;
};
memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x80000000>;
};
};
&serial2 {
......
......@@ -50,11 +50,6 @@ / {
model = "UniPhier PXs2 Vodka Board";
compatible = "socionext,uniphier-pxs2-vodka", "socionext,uniphier-pxs2";
memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x80000000>;
};
chosen {
stdout-path = "serial0:115200n8";
};
......@@ -68,6 +63,11 @@ aliases {
i2c5 = &i2c5;
i2c6 = &i2c6;
};
memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x80000000>;
};
};
&serial2 {
......
......@@ -52,12 +52,6 @@ / {
model = "UniPhier sLD3 Reference Board";
compatible = "socionext,uniphier-sld3-ref", "socionext,uniphier-sld3";
memory@8000000 {
device_type = "memory";
reg = <0x80000000 0x20000000
0xc0000000 0x20000000>;
};
chosen {
stdout-path = "serial0:115200n8";
};
......@@ -72,6 +66,12 @@ aliases {
i2c3 = &i2c3;
i2c4 = &i2c4;
};
memory@8000000 {
device_type = "memory";
reg = <0x80000000 0x20000000
0xc0000000 0x20000000>;
};
};
&ethsc {
......
......@@ -52,11 +52,6 @@ / {
model = "UniPhier sLD8 Reference Board";
compatible = "socionext,uniphier-sld8-ref", "socionext,uniphier-sld8";
memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x20000000>;
};
chosen {
stdout-path = "serial0:115200n8";
};
......@@ -71,6 +66,11 @@ aliases {
i2c2 = &i2c2;
i2c3 = &i2c3;
};
memory@80000000 {
device_type = "memory";
reg = <0x80000000 0x20000000>;
};
};
&ethsc {
......
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