On Wed, 11 Feb 2026 at 17:56, Michal Simek <[email protected]> wrote:
>
> EFI_VARIABLE_FILE_STORE is only available when FAT_WRITE is enabled but
> that's not valid for all platforms and dependency should be covered.
>
> Also Kconfig behavior is that if default option is not valid then Kconfig
> selects the first presented valid option instead hence it is better to
> record EFI_VARIABLE_NO_STORE as safe default option.
>
> Suggested-by: Tom Rini <[email protected]>
> Signed-off-by: Michal Simek <[email protected]>
> ---

Reviewed-by: Ilias Apalodimas <[email protected]>

>
> Changes in v8:
> - new patch
>
>  lib/efi_loader/Kconfig | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/lib/efi_loader/Kconfig b/lib/efi_loader/Kconfig
> index 13e44be1d067..579eed658801 100644
> --- a/lib/efi_loader/Kconfig
> +++ b/lib/efi_loader/Kconfig
> @@ -112,7 +112,8 @@ menu "UEFI Variables"
>
>  choice
>         prompt "Store for non-volatile UEFI variables"
> -       default EFI_VARIABLE_FILE_STORE
> +       default EFI_VARIABLE_FILE_STORE if FAT_WRITE
> +       default EFI_VARIABLE_NO_STORE
>         help
>           Select where non-volatile UEFI variables shall be stored.
>
> --
> 2.43.0
>

Reply via email to