[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