[PATCH] configs: socfpga: Permit EFI booting

[Chee, Tien Fong]

Hi Jan,

On 15/11/2025 1:13 am, Jan Kiszka wrote:
> [CAUTION: This email is from outside your organization. Unless you trust the sender, do not click on links or open attachments as it may be a fraudulent email attempting to steal your information and/or compromise your computer.]
>
> From: Jan Kiszka <jan.kiszka at siemens.com>
>
> Commit f369e1564cf4 turned those off because the SPL size grew too
> large. It also argued that those boards would never support EFI booting.
> The former is correct, therefore keep CONFIG_SPL_EFI_PARTITION off.
> CONFIG_SPL_ISO_PARTITION is default off. What is not correct is that
> those boards are limited to legacy boot - you just need a hybrid
> partition table to make both the bootrom and UEFI happy.
>
> Signed-off-by: Jan Kiszka <jan.kiszka at siemens.com>
> ---
>   configs/socfpga_agilex5_defconfig       | 3 +--
>   configs/socfpga_agilex_defconfig        | 3 +--
>   configs/socfpga_arria5_defconfig        | 3 +--
>   configs/socfpga_cyclone5_defconfig      | 3 +--
>   configs/socfpga_de0_nano_soc_defconfig  | 3 +--
>   configs/socfpga_de10_nano_defconfig     | 3 +--
>   configs/socfpga_de10_standard_defconfig | 3 +--
>   configs/socfpga_de1_soc_defconfig       | 3 +--
>   configs/socfpga_is1_defconfig           | 3 +--
>   configs/socfpga_mcvevk_defconfig        | 3 +--
>   configs/socfpga_secu1_defconfig         | 3 +--
>   configs/socfpga_sockit_defconfig        | 3 +--
>   configs/socfpga_socrates_defconfig      | 3 +--
>   configs/socfpga_sr1500_defconfig        | 3 +--
>   configs/socfpga_vining_fpga_defconfig   | 3 +--
>   15 files changed, 15 insertions(+), 30 deletions(-)
>
> diff --git a/configs/socfpga_agilex5_defconfig b/configs/socfpga_agilex5_defconfig
> index bbe712f8452..6347090105e 100644
> --- a/configs/socfpga_agilex5_defconfig
> +++ b/configs/socfpga_agilex5_defconfig
> @@ -67,8 +67,7 @@ CONFIG_CMD_WDT=y
>   CONFIG_CMD_CACHE=y
>   CONFIG_CMD_TIMER=y
>   CONFIG_CMD_UBI=y
> -# CONFIG_ISO_PARTITION is not set
> -# CONFIG_EFI_PARTITION is not set
> +# CONFIG_SPL_EFI_PARTITION is not set

Instead of adding this to every SoCFPGA-related defconfig, you can 
simply disable it in Kconfig. Set:

config SPL_EFI_PARTITION
default n if ARCH_SOCFPGA

as shown here: 
https://github.com/u-boot/u-boot/blob/69c8ea98a713f88f4a6ffad9df2a72b1a767f629/disk/Kconfig#L141

This approach avoids duplication across defconfigs and reduces the 
chance of missing other SoCFPGA platform variants.

Thanks.

Tien Fong