[PATCH] efi_loader: Do not enable by default on armv7

[Heinrich Schuchardt]

On 6/27/21 9:44 PM, Tom Rini wrote:
> We have had EFI loader support enabled by default on armv7 platforms for
> a long time now.  However, there has been little uptake in the wider
> community to use EFI loader support as the step to start a generic OS
> distribution, on this architecture family and the focus has been on
> armv8 instead.  Disable this by default for armv7 only.

NAK

Please, read https://fedoraproject.org/wiki/Changes/uEFIforARMv7.

Best regards

Heinrich

>
> Cc: Alexander Graf <agraf at csgraf.de>
> Cc: Heinrich Schuchardt <xypron.glpk at gmx.de>
> Cc: Ilias Apalodimas <ilias.apalodimas at linaro.org>
> Cc: Mark Kettenis <kettenis at openbsd.org>
> Cc: Masahisa Kojima <masahisa.kojima at linaro.org>
> Cc: Peter Robinson <pbrobinson at gmail.com>
> Cc: Sughosh Ganu <sughosh.ganu at linaro.org>
> Cc: Vagrant Cascadian <vagrant at debian.org>
> Signed-off-by: Tom Rini <trini at konsulko.com>
> ---
> I'm very much looking for feedback here that this is the wrong approach
> and that I've missed the off the shelf distributions that make use of
> this.
> ---
>   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 6242caceb7f9..8a22fe9951a3 100644
> --- a/lib/efi_loader/Kconfig
> +++ b/lib/efi_loader/Kconfig
> @@ -10,7 +10,7 @@ config EFI_LOADER
>   	depends on !EFI_STUB || !X86_64 || EFI_STUB_64BIT
>   	# We need EFI_STUB_32BIT to be set on x86_32 with EFI_STUB
>   	depends on !EFI_STUB || !X86 || X86_64 || EFI_STUB_32BIT
> -	default y if !ARM || SYS_CPU = armv7 || SYS_CPU = armv8
> +	default y if !ARM || SYS_CPU = armv8
>   	select LIB_UUID
>   	select HAVE_BLOCK_DEVICE
>   	select REGEX
>