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

[Heinrich Schuchardt]

On 6/27/21 11:03 PM, Heinrich Schuchardt wrote:
> 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.

OpenBSD and FreeBSD rely on UEFI for booting.

>
> 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
>>
>