From mboxrd@z Thu Jan  1 00:00:00 1970
From: Heinrich Schuchardt <xypron.glpk@gmx.de>
Date: Fri, 18 Sep 2020 19:14:02 +0200
Subject: [PATCH 1/1] efi_loader: QEMU CONFIG_EFI_GRUB_ARM32_WORKAROUND=n
In-Reply-To: <20200918112809.GR14816@bill-the-cat>
References: <20200917171808.18297-1-xypron.glpk@gmx.de>
 <20200917174732.GM14816@bill-the-cat>
 <58639675-8e19-f7f5-f24d-7bfd9fe56f03@gmx.de>
 <20200917185320.GN14816@bill-the-cat>
 <5CA71676-0903-4BA3-A1CD-BA7FA90B34BD@gmx.de>
 <20200918112809.GR14816@bill-the-cat>
Message-ID: <08dc9bc0-1bbf-46d3-abec-a11bbc1333fa@gmx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 9/18/20 1:28 PM, Tom Rini wrote:
> On Fri, Sep 18, 2020 at 06:07:42AM +0200, Heinrich Schuchardt wrote:
>> Am 17. September 2020 20:53:20 MESZ schrieb Tom Rini <trini@konsulko.com>:
>>> On Thu, Sep 17, 2020 at 08:03:24PM +0200, Heinrich Schuchardt wrote:
>>>> On 9/17/20 7:47 PM, Tom Rini wrote:
>>>>> On Thu, Sep 17, 2020 at 07:18:08PM +0200, Heinrich Schuchardt
>>> wrote:
>>>>>
>>>>>> CONFIG_EFI_GRUB_ARM32 is only needed for architectures with caches
>>> that are
>>>>>> not managed via CP15 (or for some outdated buggy versions of
>>> GRUB). It
>>>>>> makes more sense to disable the setting per architecture than per
>>> defconfig.
>>>>>>
>>>>>> Move QEMU's CONFIG_EFI_GRUB_ARM32_WORKAROUND=n from defconfig to
>>> Kconfig.
>>>>>>
>>>>>> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
>>>>>> ---
>>>>>>  configs/qemu_arm_defconfig | 1 -
>>>>>>  lib/efi_loader/Kconfig     | 1 +
>>>>>>  2 files changed, 1 insertion(+), 1 deletion(-)
>>>>>
>>>>> Why?  Are we about to move this to be a short list of "default y if
>>> ..."
>>>>> instead?  Otherwise no, this is the level of detail that should be
>>> in a
>>>>> defconfig I think and not Kconfig.
>>>>>
>>>>
>>>> Hello Tom,
>>>>
>>>> yes, in the long run I want to turn this into "default y if". But I
>>>> first need to a list of all ARCH_* which have a cache which is not
>>>> controlled via CP15.
>>>
>>> Isn't the bigger problem / list "needs to support old / buggy GRUB" ?
>>
>> There are no operating systems distributing 2021 U-Boot with 2018 GRUB.
>
> Then we should remove default y there after v2020.10 I think, and get
> the distro folks to ack/review that.  I don't think there's any cases of
> "SoC doesn't use CP15 for cache" and "SoC uses GRUB".
>

How about l2_cache_disable() in arch/arm/mach-kirkwood/cache.c?

Best regards

Heinrich