Hello, On Thu, 3 Aug 2017 08:20:20 +0900, Nobuhiro Iwamatsu wrote: > Hi, > > 2017-07-29 6:46 GMT+09:00 Thomas Petazzoni > <thomas.petazz...@free-electrons.com>: > > The linker script for SuperH brings the .ppcenv and .ppcenvr section > > of common/env_embedded.o into the .text section. However, the .ppcenv > > section is only ever filled in by env_embedded.o when > > CONFIG_SYS_USE_PPCENV is defined, but no platforms in mainline U-Boot > > use this. > > > > In addition, common/env_embedded.o is not always built (when you use > > CONFIG_ENV_IS_NOWHERE for example), which causes the following build > > failure: > > > > Fixes: > > > > LD u-boot > > /home/thomas/sh4aeb-linux-musl/bin/sh4aeb-linux-ld.bfd: cannot find > > common/env_embedded.o > > > > We fix this by no longer adding the .ppcenv and .ppcenvr sections from > > common/env_embedded.o into the .text section. > > > > Signed-off-by: Thomas Petazzoni <thomas.petazz...@free-electrons.com> > > Acked-by: Nobuhiro Iwamatsu <iwama...@nigauri.org>
Thanks for your review! Are you going to apply the patch, as you did for "arch/sh: allow building in big-endian mode" ? Thanks! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot