Hi Fabio,
Am 03.05.2017 um 02:56 schrieb Fabio Estevam:
Hi Michael,
On Sat, Apr 29, 2017 at 4:21 AM, Michael Heimpold mhei@heimpold.de wrote:
we could move the implementation in arch/arm/mach-at91/arm926ejs/cache.c to a more generic place and re-use it.
However, the warning is only generated because lib/efi_loader/efi_runtime.c calls invalidate_icache_all from efi_runtime_relocate. This is because CONFIG_EFI_LOADER is set.
I don't know if somebody using the EFI stuff for other MXS-based boards. For our Duckbill, I would simply force # CONFIG_EFI_LOADER is not set via config.
Not sure why CONFIG_EFI_LOADER is always selected for ARM.
What about doing this?
diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig index d2b6327..a2ea9c0 100644 --- a/lib/efi_loader/Kconfig +++ b/lib/efi_loader/Kconfig @@ -1,6 +1,6 @@ config EFI_LOADER bool "Support running EFI Applications in U-Boot"
depends on (ARM || X86) && OF_LIBFDT
depends on X86 && OF_LIBFDT default y help Select this option if you want to run EFI applications (like grub2)
I don't think this is a good idea since this limits EFI support for x86 only? Maybe we should just adjust the default value for MXS-based boards, something like default y if !ARCH_MXS but at the moment there seems to be no such config symbol. I'm not sure which approach is less painful, adding such a config symbol or just moving around the existing invalidate_icache_all function and have some functionality in mxs based boards which is usually not needed there. Ideally we would do both, i.e. fine-tune the config stuff and clean up common code...
Mit freundlichen Grüßen / Kind regards Michael Heimpold