Re: [PATCH v3 06/10] corstone1000: enable bootefi selftest

[Abdellatif El Khlifi <abdellatif.elkhlifi@arm.com>]

Hi Harsimran,

On Sat, Jun 27, 2026 at 03:44:17PM +0100, Harsimran Singh Tungal wrote:
> Enable `CMD_BOOTEFI_SELFTEST` in `corstone1000_defconfig` so the later
> runtime-variable selftest patch in this series can be exercised on
> Corstone-1000.
> 
> Keep `CMD_BOOTEFI_HELLO` and `CMD_POWEROFF` explicitly disabled.
> Enabling `CMD_BOOTEFI_SELFTEST` would otherwise make
> `CMD_BOOTEFI_HELLO` default to `y` and pull in `CMD_POWEROFF` on this
> PSCI-based defconfig. The intent here is to enable the selftest command
> without broadening the rest of the board command set.
> 
> Reviewed-by: Simon Glass <sjg@chromium.org>
> Signed-off-by: Harsimran Singh Tungal <harsimransingh.tungal@arm.com>
> ---
>  configs/corstone1000_defconfig | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/configs/corstone1000_defconfig b/configs/corstone1000_defconfig
> index 876b915b6a4..e16b13043ac 100644
> --- a/configs/corstone1000_defconfig
> +++ b/configs/corstone1000_defconfig
> @@ -32,6 +32,8 @@ CONFIG_SYS_PROMPT="corstone1000# "
>  # CONFIG_CMD_CONSOLE is not set
>  CONFIG_CMD_FWU_METADATA=y
>  CONFIG_CMD_BOOTZ=y
> +# CONFIG_CMD_BOOTEFI_HELLO is not set
> +CONFIG_CMD_BOOTEFI_SELFTEST=y
>  # CONFIG_CMD_XIMG is not set
>  CONFIG_CMD_GPT=y
>  CONFIG_CMD_LOADM=y
> @@ -62,6 +64,7 @@ CONFIG_SYSRESET=y
>  CONFIG_SYSRESET_PSCI=y
>  CONFIG_TEE=y
>  CONFIG_OPTEE=y
> +# CONFIG_CMD_POWEROFF is not set
>  CONFIG_USB=y
>  CONFIG_USB_ISP1760=y
>  # CONFIG_RANDOM_UUID is not set
> -- 
> 2.34.1
> 

Reviewed-by: Abdellatif El Khlifi <abdellatif.elkhlifi@arm.com>

Kind regards