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

Mark Kettenis mark.kettenis at xs4all.nl
Thu Oct 19 11:39:28 UTC 2017


> From: Maxime Ripard <maxime.ripard at free-electrons.com>
> Date: Thu, 19 Oct 2017 10:26:49 +0200
> 
> 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.

This is an essential feature for booting OpenBSD.  And I believe it is
a requirement for several Linux distros as well.  I don't think
disabling this by default is a good idea.

How much of that 31kB is due to recent improvements of the EFI loader
support?  I understand the desire to have a more complete EFI
implementation, but if the consequence of that is that the EFI loader
gets disabled by default on many boards I think we're throwing out the
baby with the bathwater...

> 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
> -- 
> 2.14.2
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot
> 


More information about the U-Boot mailing list