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 <s...@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
