[PATCH 1/1] Added arm64 assembly for examples/api crt0

[Heinrich Schuchardt]

On 6/4/24 05:34, Brunham, Kalen wrote:
> Hi Tom/Heinrich,
>
> I have https://github.com/u-boot/u-boot/pull/574 that has a proposed change. It is currently failing clang sandbox, which looks a little strange and I'm looking into it. Please let me know if you have any feedback on the general strategy.
>
> Thanks,
> -Kalen
>
> -----Original Message-----
> From: Tom Rini <trini at konsulko.com>
> Sent: Monday, June 3, 2024 1:23 PM
> To: Brunham, Kalen <kalen.brunham at intel.com>
> Cc: Jiaxun Yang <jiaxun.yang at flygoat.com>; Heinrich Schuchardt <xypron.glpk at gmx.de>; Ilias Apalodimas <ilias.apalodimas at linaro.org>; U-Boot at lists.denx.de
> Subject: Re: [PATCH 1/1] Added arm64 assembly for examples/api crt0
>
> On Sun, Jun 02, 2024 at 12:16:38AM +0000, Brunham, Kalen wrote:
>> Thanks Tom.
>>
>> Can you suggest who is the existing owner of this code I could work with?
>
> You should CC Heinrich for sure once you have things working.
>
>>
>> -----Original Message-----
>> From: Tom Rini <trini at konsulko.com>
>> Sent: Friday, May 31, 2024 12:03 PM
>> To: Brunham, Kalen <kalen.brunham at intel.com>
>> Cc: Jiaxun Yang <jiaxun.yang at flygoat.com>; Heinrich Schuchardt <xypron.glpk at gmx.de>; Ilias Apalodimas <ilias.apalodimas at linaro.org>; U-Boot at lists.denx.de
>> Subject: Re: [PATCH 1/1] Added arm64 assembly for examples/api crt0
>>
>> On Thu, May 30, 2024 at 02:53:17PM +0000, Brunham, Kalen wrote:
>>
>>> Hi Tom,
>>>
>>> What are next steps on this change?
>>>
>>> I'm committed to the idea of EFI apps opposed to examples/standalone.
>>> EFI currently requires block devices which is not enabled when only using flash.
>>>
>>> Should config BLK be updated to include DM_SPI_FLASH?

The UEFI implementation should not depend on DM_SPI_FLASH.

You could make BLK manually selectable again instead:

diff --git a/drivers/block/Kconfig b/drivers/block/Kconfig
index 6ad18889f61..1f0e1010f0e 100644
--- a/drivers/block/Kconfig
+++ b/drivers/block/Kconfig
@@ -1,5 +1,5 @@
  config BLK
-       bool # "Support block devices"
+       bool "Support block devices"

Cf. 0417b8523c47 ("blk: Hide the BLK and SPL_LEGACY_BLOCK options")

Best regards

Heinrich

>>> Should we explore how to remove BLK as a dependency for EFI_LOADER?
>>
>> Yes, someone needs to explore how to cleanly make BLK optional for EFI_LOADER.
>>
>> --
>> Tom
>