[U-Boot] [PATCH 21/48] efi: Drop CONFIG_SYS_TEXT_BASE for EFI

Bin Meng bmeng.cn at gmail.com
Thu Jul 23 10:33:48 CEST 2015


Hi Simon,

On Wed, Jul 22, 2015 at 11:49 PM, Simon Glass <sjg at chromium.org> wrote:
> From: Ben Stoltz <stoltz at google.com>
>
> When U-Boot runs as an EFI application is does not have a definition of
> CONFIG_SYS_TEXT_BASE. U-Boot is a relocatable application and the relocation
> is done by EFI. U-Boot can be loaded at any address.
>
> Ensure that this CONFIG option is not set in this case.
>
> Signed-off-by: Ben Stoltz <stoltz at google.com>
> Signed-off-by: Simon Glass <sjg at chromium.org>

Except the name of ARCH_EFI per my previous comments:

Reviewed-by: Bin Meng <bmeng.cn at gmail.com>

> ---
>
>  Kconfig | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/Kconfig b/Kconfig
> index 15e15af..c1e7d22 100644
> --- a/Kconfig
> +++ b/Kconfig
> @@ -179,6 +179,7 @@ config SYS_EXTRA_OPTIONS
>
>  config SYS_TEXT_BASE
>         depends on SPARC || ARC || X86 || ARCH_UNIPHIER
> +       depends on !ARCH_EFI
>         hex "Text Base"
>         help
>           TODO: Move CONFIG_SYS_TEXT_BASE for all the architecture
> --

Regards,
Bin


More information about the U-Boot mailing list