[U-Boot] [PATCH 3/3] efi_loader: Do not enable it by default for sunxi

Alexander Graf agraf at suse.de
Thu Oct 19 08:51:04 UTC 2017


On 10/19/2017 10:26 AM, Maxime Ripard wrote:
> The EFI loader support takes around 31kB on an ARMv7 board, which makes us
> trip across the size limit we've had on the U-Boot binary.
>
> Since it's not an essential feature, disable it by default for ARCH_SUNXI
> so that we get back some extra room for user customisations.
>
> Signed-off-by: Maxime Ripard <maxime.ripard at free-electrons.com>

Quite the contrary - it is essential. All major distributions are going 
for distro boot + EFI at least for 64bit platforms now. Disabling it by 
default means you basically kill your user base on those.

> ---
>   lib/efi_loader/Kconfig | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig
> index d2b6327119b4..a80a914b2fe8 100644
> --- a/lib/efi_loader/Kconfig
> +++ b/lib/efi_loader/Kconfig
> @@ -1,7 +1,7 @@
>   config EFI_LOADER
>   	bool "Support running EFI Applications in U-Boot"
>   	depends on (ARM || X86) && OF_LIBFDT
> -	default y
> +	default y if !ARCH_SUNXI

Nack on any change to that default line. If you must disable efi_loader 
(and really, I strongly advise not to do so for sunxi), please do so in 
the defconfigs, as nothing prohibits the architecture to work with it.


Alex



More information about the U-Boot mailing list