Re: [PATCH 1/1] efi: turn on EFI_DEBUG_SUPPORT for qemu arm and arm64 target

[Heinrich Schuchardt]

Am 7. Juli 2025 11:53:22 MESZ schrieb "Ying-Chun Liu (PaulLiu)" <paulliu at debian.org>:
>Previously, EFI_DEBUG_SUPPORT was enabled across all platforms for
>CI build testing. With this commit, we are limiting its activation
>to qemu_arm64_defconfig and qemu_arm_defconfig only, which is
>sufficient for our current needs.

Was there a request to turn it off on other platforms?

Best regards 

Heinrich


>
>Signed-off-by: Ying-Chun Liu (PaulLiu) <paulliu at debian.org>
>Cc: Heinrich Schuchardt <xypron.glpk at gmx.de>
>Cc: Ilias Apalodimas <ilias.apalodimas at linaro.org>
>Cc: Tom Rini <trini at konsulko.com>
>---
> configs/qemu_arm64_defconfig | 1 +
> configs/qemu_arm_defconfig   | 1 +
> lib/efi_loader/Kconfig       | 1 -
> 3 files changed, 2 insertions(+), 1 deletion(-)
>
>diff --git a/configs/qemu_arm64_defconfig b/configs/qemu_arm64_defconfig
>index 39afb837e41..54bf4646382 100644
>--- a/configs/qemu_arm64_defconfig
>+++ b/configs/qemu_arm64_defconfig
>@@ -15,6 +15,7 @@ CONFIG_ENV_ADDR=0x4000000
> CONFIG_PCI=y
> CONFIG_DEBUG_UART=y
> CONFIG_AHCI=y
>+CONFIG_EFI_DEBUG_SUPPORT=y
> CONFIG_EFI_HTTP_BOOT=y
> CONFIG_FIT=y
> CONFIG_FIT_SIGNATURE=y
>diff --git a/configs/qemu_arm_defconfig b/configs/qemu_arm_defconfig
>index 92ba48f6af9..b1c277a86c0 100644
>--- a/configs/qemu_arm_defconfig
>+++ b/configs/qemu_arm_defconfig
>@@ -18,6 +18,7 @@ CONFIG_ENV_ADDR=0x4000000
> CONFIG_PCI=y
> CONFIG_DEBUG_UART=y
> CONFIG_AHCI=y
>+CONFIG_EFI_DEBUG_SUPPORT=y
> CONFIG_FIT=y
> CONFIG_FIT_SIGNATURE=y
> CONFIG_FIT_VERBOSE=y
>diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig
>index 077466f01f0..fdabe9cc278 100644
>--- a/lib/efi_loader/Kconfig
>+++ b/lib/efi_loader/Kconfig
>@@ -73,7 +73,6 @@ config EFI_SIGNATURE_SUPPORT
> 
> config EFI_DEBUG_SUPPORT
> 	bool "EFI Debug Support"
>-	default y if !HAS_BOARD_SIZE_LIMIT
> 	help
> 	  Select this option if you want to setup the EFI Debug Support
> 	  Table and the EFI_SYSTEM_TABLE_POINTER which is used by the debug