[U-Boot] arm: mxs: No arch specific invalidate_icache_all available!

[Michael Heimpold]

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 at 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
-- 
Software Engineer

I2SE GmbH                           Tel: +49 (0) 341 355667-00
Friedrich-Ebert-Str. 61             Fax: +49 (0) 341 355667-02
04109 Leipzig
Germany
Web: http://www.i2se.com/           Mail: info at i2se.com
VAT No.: DE 811528334
Amtsgericht Leipzig HRB 23784
Geschäftsführer/CEO: Carsten Ziermann

*** Diese E-Mail ist allein für den bezeichneten Adressaten bestimmt. Sie kann rechtlich vertrauliche Informationen enthalten. Wenn Sie diese E-Mail irrtümlich erhalten haben, informieren Sie bitte unverzüglich den Absender per E-Mail und löschen Sie diese E-Mail von Ihrem Computer, ohne Kopien anzufertigen.
Vielen Dank. ***

*** This email is for the exclusive use of the addressee. It may contain legally privileged information. If you have received this message in error, please notify the sender by email immediately and delete the message from your computer without making any copies.
Thank you. ***