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 <[email protected]>
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 <[email protected]>
---
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
