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

[Tom Rini]

On Tue, Jun 04, 2024 at 07:57:38AM +0200, Heinrich Schuchardt wrote:
> 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")

They're hidden on purpose because most things should select them, not
depend on them. If EFI_LOADER is making its own virtual block devices,
it should be select'ing it not depending on it. There's a few other
places getting this backwards I see.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20240604/4afe9e7d/attachment.sig>