[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