[U-Boot] Build system: Don't check for CONFIG_SYS_TEXT_BASE being set

Tom Rini trini at konsulko.com
Sun Feb 11 20:47:33 UTC 2018


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


More information about the U-Boot mailing list