On 01.07.25 05:49, Ying-Chun Liu (PaulLiu) wrote:
From: "Ying-Chun Liu (PaulLiu)" <paul....@linaro.org>

Enable CONFIG_EFI_DEBUG_SUPPORT so that we can run a build test for
this config.

It is not enough to build only on 64bit. qemu_arm_defconfig showed this problem:

lib/efi_loader/efi_debug_support.c: In function ‘efi_initialize_system_table_pointer’: lib/efi_loader/efi_debug_support.c:48:49: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] 48 | systab_pointer->efi_system_table_base = (efi_physical_addr_t)&systab;
      |

Either enable in multiple defconfigs or maybe use

    default y if !HAS_BOARD_SIZE_LIMIT

in the Kconfig file.

Best regards

Heinrich


Signed-off-by: Ying-Chun Liu (PaulLiu) <paul....@linaro.org>
Cc: Simon Glass <s...@chromium.org>
Cc: Heinrich Schuchardt <xypron.g...@gmx.de>
Cc: Ilias Apalodimas <ilias.apalodi...@linaro.org>
---
V7: enable CONFIG_EFI_DEBUG_SUPPORT for sandbox_defconfig
---
  configs/sandbox_defconfig | 1 +
  1 file changed, 1 insertion(+)

diff --git a/configs/sandbox_defconfig b/configs/sandbox_defconfig
index 4f6943d1a4b..c50746d3dbc 100644
--- a/configs/sandbox_defconfig
+++ b/configs/sandbox_defconfig
@@ -10,6 +10,7 @@ CONFIG_PCI=y
  CONFIG_DEBUG_UART=y
  CONFIG_SYS_MEMTEST_START=0x00100000
  CONFIG_SYS_MEMTEST_END=0x00101000
+CONFIG_EFI_DEBUG_SUPPORT=y
  CONFIG_EFI_SECURE_BOOT=y
  CONFIG_EFI_RT_VOLATILE_STORE=y
  CONFIG_EFI_RUNTIME_UPDATE_CAPSULE=y

Reply via email to