2018-01-31 0:23 GMT+09:00 Alexey Brodkin <alexey.brod...@synopsys.com>: > 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> > --- > 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
Please feel free to include historical reason: This ifneq conditional was added for Blackfin because it did not use CONFIG_SYS_TEXT_BASE. See. https://github.com/u-boot/u-boot/blob/v2017.03/arch/blackfin/config.mk#L67 Then Blackfin is gone, so should be OK. Acked-by: Masahiro Yamada <yamada.masah...@socionext.com> > # Normally we fill empty space with 0xff > quiet_cmd_objcopy = OBJCOPY $@ > -- > 2.14.3 > > _______________________________________________ > U-Boot mailing list > U-Boot@lists.denx.de > https://lists.denx.de/listinfo/u-boot -- Best Regards Masahiro Yamada _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot