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 abrodkin@synopsys.com Cc: Tom Rini trini@konsulko.com Acked-by: Masahiro Yamada yamada.masahiro@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.