On 10/17/23 00:27, Simon Glass wrote:
Add this to some more commands to avoid build errors with sandbox.
Note that this is a temporary solution to expose more problems. A later patch puts these behind an 'if CMDLINE'
Signed-off-by: Simon Glass sjg@chromium.org
Changes in v3:
Add an explation as to why this patch is here
cmd/Kconfig | 6 ++++++ 1 file changed, 6 insertions(+)
diff --git a/cmd/Kconfig b/cmd/Kconfig index 5bc0a92d57ad..00a7f84bce99 100644 --- a/cmd/Kconfig +++ b/cmd/Kconfig @@ -231,6 +231,7 @@ menu "Boot commands"
config CMD_BOOTD bool "bootd"
- depends on CMDLINE
Why don't we simply use a single "if" for all commands? Adding the same dependency to 200+ commands does not feel right.
If there is any CONFIG_CMD* that is needed without CMDLINE, then that code should move to lib/ or drivers/.
Best regards
Heinrich
default y help Run the command stored in the environment "bootcmd", i.e. @@ -420,6 +421,7 @@ source lib/efi_selftest/Kconfig
config CMD_BOOTMENU bool "bootmenu"
- depends on CMDLINE select MENU select CHARSET help
@@ -486,6 +488,7 @@ config CMD_GO
config CMD_RUN bool "run"
- depends on CMDLINE default y help Run the command in the given environment variable.
@@ -576,6 +579,7 @@ menu "Environment commands"
config CMD_ASKENV bool "ask for env variable"
- depends on CMDLINE help Ask for environment variable
@@ -2132,6 +2136,7 @@ config CMD_EFICONFIG
config CMD_EXCEPTION bool "exception - raise exception"
- depends on CMDLINE depends on ARM || RISCV || SANDBOX || X86 help Enable the 'exception' command which allows to raise an exception.
@@ -2238,6 +2243,7 @@ config CMD_SYSBOOT
config CMD_QFW bool "qfw"
- depends on CMDLINE select QFW help This provides access to the QEMU firmware interface. The main