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

[Tom Rini]

On Thu, Oct 19, 2017 at 10:26:49AM +0200, 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>
> ---
>  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
>  	help
>  	  Select this option if you want to run EFI applications (like grub2)
>  	  on top of U-Boot. If this option is enabled, U-Boot will expose EFI

I want to speak against this particular option.  "U-Boot boots EFI
application, $distro just works" is a huge deal right now.  As much as I
would have preferred various other things happen at various points in
the past, kicking off an EFI application to boot Linux is huge and
important outside of the embedded space (and ARM Ltd is pushing this
path within the embedded space).  There are cases where we don't want
the EFI loader, but for ARMv7/AArch64 this should be enabled by default
and turned off at the board level for specific use cases.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171019/e09517c3/attachment.sig>