On Tue, Jan 30, 2018 at 06:23:13PM +0300, Alexey Brodkin wrote: > CONFIG_SYS_TEXT_BASE must be set anyways and then it is used in many > places in the same Makefile without any checks so there's no point in > keeping this check araound just in one place. > > Signed-off-by: Alexey Brodkin <abrod...@synopsys.com> > Cc: Tom Rini <tr...@konsulko.com> > Acked-by: Masahiro Yamada <yamada.masah...@socionext.com> > --- > Makefile | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/Makefile b/Makefile > index ab3453dcebdc..6f15612b4d07 100644 > --- a/Makefile > +++ b/Makefile > @@ -820,9 +820,7 @@ LDFLAGS_u-boot += $(LDFLAGS_FINAL) > # Avoid 'Not enough room for program headers' error on binutils 2.28 onwards. > LDFLAGS_u-boot += $(call ld-option, --no-dynamic-linker) > > -ifneq ($(CONFIG_SYS_TEXT_BASE),) > LDFLAGS_u-boot += -Ttext $(CONFIG_SYS_TEXT_BASE) > -endif
This then causes xtensa to fail to build as it does not set CONFIG_SYS_TEXT_BASE. -- Tom _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
