Hi Simon, No news about fdtgrep?
I will apply this patch in a few days, though I am not happy about it. 2016-03-28 22:10 GMT+09:00 Masahiro Yamada <[email protected]>: > Currently, fdtgrep would not accept uniphier-ph1-ld20-ref.dtb > and uniphier-ph1-ld11-ref.dtb unless the aliases node comes > the first in the root node. > > $ make -s uniphier_pxs2_ld6b_defconfig > $ make CROSS_COMPILE=arm-linux-gnueabi- DEVICE_TREE=uniphier-ph1-ld20-ref > [snip] > LDS spl/u-boot-spl.lds > LD spl/u-boot-spl > OBJCOPY spl/u-boot-spl-nodtb.bin > FDTGREP spl/u-boot-spl.dtb > Error at 'fdt_find_regions': FDT_ERR_BADLAYOUT > /aliases node must come before all other nodes > Error: FDT_ERR_BADMAGIC > make[1]: *** [spl/u-boot-spl.dtb] Error 1 > make: *** [spl/u-boot-spl] Error 2 > > This commit moves the aliases node as the error message from the > fdtgrep tool suggests, although this requirement does not sound > reasonable to me. > > Signed-off-by: Masahiro Yamada <[email protected]> > --- > > Simon, > > I'd like to avoid this patch if possible. > > Not figured out the root cause of the problem yet? > > > > arch/arm/dts/uniphier-ph1-ld11-ref.dts | 18 +++++++++--------- > arch/arm/dts/uniphier-ph1-ld20-ref.dts | 18 +++++++++--------- > 2 files changed, 18 insertions(+), 18 deletions(-) > > diff --git a/arch/arm/dts/uniphier-ph1-ld11-ref.dts > b/arch/arm/dts/uniphier-ph1-ld11-ref.dts > index a624a49..88e7f53 100644 > --- a/arch/arm/dts/uniphier-ph1-ld11-ref.dts > +++ b/arch/arm/dts/uniphier-ph1-ld11-ref.dts > @@ -14,15 +14,6 @@ > model = "UniPhier PH1-LD11 Reference Board"; > compatible = "socionext,ph1-ld11-ref", "socionext,ph1-ld11"; > > - memory { > - device_type = "memory"; > - reg = <0 0x80000000 0 0x40000000>; > - }; > - > - chosen { > - stdout-path = "serial0:115200n8"; > - }; > - > aliases { > serial0 = &serial0; > serial1 = &serial1; > @@ -35,6 +26,15 @@ > i2c4 = &i2c4; > i2c5 = &i2c5; > }; > + > + memory { > + device_type = "memory"; > + reg = <0 0x80000000 0 0x40000000>; > + }; > + > + chosen { > + stdout-path = "serial0:115200n8"; > + }; > }; > > ðsc { > diff --git a/arch/arm/dts/uniphier-ph1-ld20-ref.dts > b/arch/arm/dts/uniphier-ph1-ld20-ref.dts > index 108adeb..a55d90c 100644 > --- a/arch/arm/dts/uniphier-ph1-ld20-ref.dts > +++ b/arch/arm/dts/uniphier-ph1-ld20-ref.dts > @@ -14,15 +14,6 @@ > model = "UniPhier PH1-LD20 Reference Board"; > compatible = "socionext,ph1-ld20-ref", "socionext,ph1-ld20"; > > - memory { > - device_type = "memory"; > - reg = <0 0x80000000 0 0xc0000000>; > - }; > - > - chosen { > - stdout-path = "serial0:115200n8"; > - }; > - > aliases { > serial0 = &serial0; > serial1 = &serial1; > @@ -35,6 +26,15 @@ > i2c4 = &i2c4; > i2c5 = &i2c5; > }; > + > + memory { > + device_type = "memory"; > + reg = <0 0x80000000 0 0xc0000000>; > + }; > + > + chosen { > + stdout-path = "serial0:115200n8"; > + }; > }; > > ðsc { > -- > 1.9.1 > > _______________________________________________ > U-Boot mailing list > [email protected] > http://lists.denx.de/mailman/listinfo/u-boot -- Best Regards Masahiro Yamada _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
