public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [PATCH 1/1] cmd: CONFIG_CMD_EFICONFIG requires CONFIG_MENU
@ 2023-02-08 13:01 Heinrich Schuchardt
  2023-02-08 16:25 ` Simon Glass
  2023-02-09  9:28 ` Ilias Apalodimas
  0 siblings, 2 replies; 3+ messages in thread
From: Heinrich Schuchardt @ 2023-02-08 13:01 UTC (permalink / raw)
  To: Ilias Apalodimas; +Cc: u-boot, Heinrich Schuchardt

The eficonfig command invokes functions implemented in common/menu.c like

* menu_default_set()
* menu_get_choice()
* menu_destroy()
* menu_item_add()

Fixes: 87d791423ac6 ("eficonfig: menu-driven addition of UEFI boot option")
Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
---
 cmd/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/cmd/Kconfig b/cmd/Kconfig
index c7344ee1f6..aef99d2eb8 100644
--- a/cmd/Kconfig
+++ b/cmd/Kconfig
@@ -2001,6 +2001,7 @@ config CMD_EFIDEBUG
 config CMD_EFICONFIG
 	bool "eficonfig - provide menu-driven uefi variables maintenance interface"
 	depends on CMD_BOOTEFI_BOOTMGR
+	select MENU
 	help
 	  Enable the 'eficonfig' command which provides the menu-driven UEFI
 	  variable maintenance interface.
-- 
2.38.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/1] cmd: CONFIG_CMD_EFICONFIG requires CONFIG_MENU
  2023-02-08 13:01 [PATCH 1/1] cmd: CONFIG_CMD_EFICONFIG requires CONFIG_MENU Heinrich Schuchardt
@ 2023-02-08 16:25 ` Simon Glass
  2023-02-09  9:28 ` Ilias Apalodimas
  1 sibling, 0 replies; 3+ messages in thread
From: Simon Glass @ 2023-02-08 16:25 UTC (permalink / raw)
  To: Heinrich Schuchardt; +Cc: Ilias Apalodimas, u-boot

On Wed, 8 Feb 2023 at 06:02, Heinrich Schuchardt
<heinrich.schuchardt@canonical.com> wrote:
>
> The eficonfig command invokes functions implemented in common/menu.c like
>
> * menu_default_set()
> * menu_get_choice()
> * menu_destroy()
> * menu_item_add()
>
> Fixes: 87d791423ac6 ("eficonfig: menu-driven addition of UEFI boot option")
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
> ---
>  cmd/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
>

Reviewed-by: Simon Glass <sjg@chromium.org>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 1/1] cmd: CONFIG_CMD_EFICONFIG requires CONFIG_MENU
  2023-02-08 13:01 [PATCH 1/1] cmd: CONFIG_CMD_EFICONFIG requires CONFIG_MENU Heinrich Schuchardt
  2023-02-08 16:25 ` Simon Glass
@ 2023-02-09  9:28 ` Ilias Apalodimas
  1 sibling, 0 replies; 3+ messages in thread
From: Ilias Apalodimas @ 2023-02-09  9:28 UTC (permalink / raw)
  To: Heinrich Schuchardt; +Cc: u-boot

On Wed, Feb 08, 2023 at 02:01:33PM +0100, Heinrich Schuchardt wrote:
> The eficonfig command invokes functions implemented in common/menu.c like
> 
> * menu_default_set()
> * menu_get_choice()
> * menu_destroy()
> * menu_item_add()
> 
> Fixes: 87d791423ac6 ("eficonfig: menu-driven addition of UEFI boot option")
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
> ---
>  cmd/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/cmd/Kconfig b/cmd/Kconfig
> index c7344ee1f6..aef99d2eb8 100644
> --- a/cmd/Kconfig
> +++ b/cmd/Kconfig
> @@ -2001,6 +2001,7 @@ config CMD_EFIDEBUG
>  config CMD_EFICONFIG
>  	bool "eficonfig - provide menu-driven uefi variables maintenance interface"
>  	depends on CMD_BOOTEFI_BOOTMGR
> +	select MENU
>  	help
>  	  Enable the 'eficonfig' command which provides the menu-driven UEFI
>  	  variable maintenance interface.
> -- 
> 2.38.1
> 
Reviewed-by: Ilias Apalodimas <ilias.apalodimas@linaro.org>


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-02-09  9:29 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-02-08 13:01 [PATCH 1/1] cmd: CONFIG_CMD_EFICONFIG requires CONFIG_MENU Heinrich Schuchardt
2023-02-08 16:25 ` Simon Glass
2023-02-09  9:28 ` Ilias Apalodimas

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox