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

[Brunham, Kalen]

I can see that efi_device_path.c and efi_disk.c both #include blk.h. 

-----Original Message-----
From: Tom Rini <trini at konsulko.com> 
Sent: Thursday, May 23, 2024 11:33 AM
To: Brunham, Kalen <kalen.brunham at intel.com>; Heinrich Schuchardt <xypron.glpk at gmx.de>
Cc: U-Boot at lists.denx.de
Subject: Re: [PATCH 1/1] Added arm64 assembly for examples/api crt0

On Wed, May 22, 2024 at 05:22:24PM +0000, Brunham, Kalen wrote:

> Hi Tom,
> 
> BLK is currently a dependency for EFI_LOADER as shown in the snippet from lib/efi_loader/Kconfig below. Perhaps the question is why EFI_LOADER requires a block device? If I remove this depends on BLK line, then I can enable EFI and successfully simulate the EFI hello world on my test design. 
> 
> 
> config EFI_LOADER
> 	bool "Support running UEFI applications"
> 	depends on OF_LIBFDT && ( \
> 		ARM && (SYS_CPU = arm1136 || \
> 			SYS_CPU = arm1176 || \
> 			SYS_CPU = armv7   || \
> 			SYS_CPU = armv8)  || \
> 		X86 || RISCV || SANDBOX)
> 	# We need EFI_STUB_64BIT to be set on x86_64 with EFI_STUB
> 	depends on !EFI_STUB || !X86_64 || EFI_STUB_64BIT
> 	# We need EFI_STUB_32BIT to be set on x86_32 with EFI_STUB
> 	depends on !EFI_STUB || !X86 || X86_64 || EFI_STUB_32BIT
> 	depends on BLK
> 	depends on !EFI_APP
> 	default y if !ARM || SYS_CPU = armv7 || SYS_CPU = armv8

Do you recall why this is Heinrich?

-- 
Tom