Re: [PATCH v3 2/2] spl: Add support for booting from ESP

[Tom Rini <trini@konsulko.com>]

[-- Attachment #1: Type: text/plain, Size: 1425 bytes --]

On Mon, Jan 22, 2024 at 04:40:22PM +0530, Mayuresh Chitale wrote:

> Some platforms as described by EBBR specification may store images in
> the FIRMWARE directory of the UEFI system partition(ESP). Add support
> to boot from the EFI system partition if it is enabled for a platform.
> 
> Signed-off-by: Mayuresh Chitale <mchitale@ventanamicro.com>
> Reviewed-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
[snip]
> diff --git a/common/spl/Kconfig b/common/spl/Kconfig
> index cf7ffc9b112..48e4e43196a 100644
> --- a/common/spl/Kconfig
> +++ b/common/spl/Kconfig
> @@ -1292,6 +1292,14 @@ config SPL_SATA_RAW_U_BOOT_SECTOR
>  	  Sector on the SATA disk to load U-Boot from, when the SATA disk is being
>  	  used in raw mode. Units: SATA disk sectors (1 sector = 512 bytes).
>  
> +config SPL_ESP_BOOT
> +	bool "Load next stage boot image from the UEFI system partition"
> +	default y if BOOT_DEFAULTS
> +	select SPL_PARTITION_TYPE_GUID
> +	help
> +	  When enabled, first try to boot from the UEFI system partition as
> +	  described in the Ch.4 of the EBBR specification.
> +
>  config SPL_NVME
>  	bool "NVM Express device support"
>  	depends on BLK

This introduces:
WARNING: unmet direct dependencies detected for SPL_PARTITION_TYPE_GUID
  Depends on [n]: SPL_EFI_PARTITION [=n]
  Selected by [y]:
  - SPL_ESP_BOOT [=y] && SPL [=y]

On for example coreboot64.

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]