2018-01-31 0:23 GMT+09:00 Alexey Brodkin Alexey.Brodkin@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 abrodkin@synopsys.com Cc: Tom Rini trini@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.masahiro@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