On Tue, Nov 19, 2019 at 04:35:57AM +0100, Heinrich Schuchardt wrote:
> With an implementation for allow_unaligned() available for arm1136 and
> arm1176 UEFI can be supported on these architectures.
> 
> Signed-off-by: Heinrich Schuchardt <[email protected]>
> ---
>  lib/efi_loader/Kconfig | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig
> index 2f40e485ef..7984d6f42d 100644
> --- a/lib/efi_loader/Kconfig
> +++ b/lib/efi_loader/Kconfig
> @@ -1,8 +1,10 @@
>  config EFI_LOADER
>       bool "Support running UEFI applications"
>       depends on OF_LIBFDT && ( \
> -             ARM && (SYS_CPU = armv7 || \
> -                     SYS_CPU = armv8) || \
> +             ARM && (SYS_CPU = arm1136 || \
> +                     SYS_CPU = arm1176 || \
> +                     SYS_CPU = armv7   || \
> +                     SYS_CPU = armv8)  || \
>               X86 || RISCV || SANDBOX)
>       # We need EFI_STUB_64BIT to be set on x86_64 with EFI_STUB
>       depends on !EFI_STUB || !X86_64 || EFI_STUB_64BIT

My concern is that while it's good and important we're correcting the
dependencies for ARM, non-ARMv7 hardware is much more legacy than new
design and we maybe don't want to have it enabled by default there.  One
of the biggest pieces of feedback I have gotten over the overall EFI
support is that it being on by default on legacy systems causes a lot of
unwanted size growth.  Does Kconfig work sanely with something like:
        default y if !ARM
        default y if ARM && (SYS_CPU = armv7 || SYS_CPU == arv8)
?  Then we could allow other older ARM systems to enable it still and
now function (again, good work) without the size growth by default.
Thanks!

-- 
Tom

Attachment: signature.asc
Description: PGP signature

_______________________________________________
U-Boot mailing list
[email protected]
https://lists.denx.de/listinfo/u-boot

Reply via email to