Commit b9f2fc38 authored by Masahiro Yamada's avatar Masahiro Yamada

arm64: 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 b5027603
...@@ -52,11 +52,6 @@ / { ...@@ -52,11 +52,6 @@ / {
model = "UniPhier LD11 Reference Board"; model = "UniPhier LD11 Reference Board";
compatible = "socionext,uniphier-ld11-ref", "socionext,uniphier-ld11"; compatible = "socionext,uniphier-ld11-ref", "socionext,uniphier-ld11";
memory@80000000 {
device_type = "memory";
reg = <0 0x80000000 0 0x40000000>;
};
chosen { chosen {
stdout-path = "serial0:115200n8"; stdout-path = "serial0:115200n8";
}; };
...@@ -73,6 +68,11 @@ aliases { ...@@ -73,6 +68,11 @@ aliases {
i2c4 = &i2c4; i2c4 = &i2c4;
i2c5 = &i2c5; i2c5 = &i2c5;
}; };
memory@80000000 {
device_type = "memory";
reg = <0 0x80000000 0 0x40000000>;
};
}; };
&ethsc { &ethsc {
......
...@@ -52,11 +52,6 @@ / { ...@@ -52,11 +52,6 @@ / {
model = "UniPhier LD20 Reference Board"; model = "UniPhier LD20 Reference Board";
compatible = "socionext,uniphier-ld20-ref", "socionext,uniphier-ld20"; compatible = "socionext,uniphier-ld20-ref", "socionext,uniphier-ld20";
memory@80000000 {
device_type = "memory";
reg = <0 0x80000000 0 0xc0000000>;
};
chosen { chosen {
stdout-path = "serial0:115200n8"; stdout-path = "serial0:115200n8";
}; };
...@@ -73,6 +68,11 @@ aliases { ...@@ -73,6 +68,11 @@ aliases {
i2c4 = &i2c4; i2c4 = &i2c4;
i2c5 = &i2c5; i2c5 = &i2c5;
}; };
memory@80000000 {
device_type = "memory";
reg = <0 0x80000000 0 0xc0000000>;
};
}; };
&ethsc { &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