[PATCH] bootstd: Correct default boot command

Simon Glass sjg at chromium.org
Sat May 6 16:27:09 CEST 2023

The patch to relax flag requirements was not accepted[1], so we still have
to have separate bootcommands depending on CMD_BOOTFLOW_FULL.

The previous attempt at this did not work, since it used the wrong name
for the options.

Fix this and change the message to mention BOOTSTD_FULL since this affects
not just the flags, but all functionality, so is more likely what the user

Drop the useless condition on CMD_BOOTFLOW_FULL while we are here.

[1] https://patchwork.ozlabs.org/project/uboot/patch/20230329071655.1959513-2-sjg@chromium.org/

Signed-off-by: Simon Glass <sjg at chromium.org>
Fixes: a91492b6e9c ("bootstd: Provide a default command")

 boot/Kconfig   | 4 ++--
 cmd/Kconfig    | 2 +-
 cmd/bootflow.c | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/boot/Kconfig b/boot/Kconfig
index 9882812eecb8..fad8e530b17f 100644
--- a/boot/Kconfig
+++ b/boot/Kconfig
@@ -1554,8 +1554,8 @@ config USE_BOOTCOMMAND
 	string "bootcmd value"
-	default "bootflow scan -lb" if BOOTSTD_BOOTCOMMAND && CMD_BOOTFLOW_FULL
-	default "bootflow scan" if BOOTSTD_BOOTCOMMAND && !CMD_BOOTFLOW_FULL
+	default "bootflow scan -lb" if BOOTSTD_DEFAULTS && CMD_BOOTFLOW_FULL
+	default "bootflow scan" if BOOTSTD_DEFAULTS && !CMD_BOOTFLOW_FULL
 	default "run distro_bootcmd" if !BOOTSTD_BOOTCOMMAND && DISTRO_DEFAULTS
 	  This is the string of commands that will be used as bootcmd and if
diff --git a/cmd/Kconfig b/cmd/Kconfig
index e45b8847aef7..a630a343e25d 100644
--- a/cmd/Kconfig
+++ b/cmd/Kconfig
@@ -258,7 +258,7 @@ config CMD_BOOTFLOW
 	bool "bootflow - extract subcommands"
 	depends on BOOTSTD_FULL
-	default y if BOOTSTD_FULL
+	default y
 	  Add the ability to list the available bootflows, select one and obtain
 	  information about it.
diff --git a/cmd/bootflow.c b/cmd/bootflow.c
index cfe342269896..6cc58e48c7d0 100644
--- a/cmd/bootflow.c
+++ b/cmd/bootflow.c
@@ -125,7 +125,7 @@ static int do_bootflow_scan(struct cmd_tbl *cmdtp, int flag, int argc,
 			dev = std->cur_bootdev;
 	} else {
 		if (has_args) {
-			printf("Flags not supported: enable CONFIG_BOOTFLOW_FULL\n");
+			printf("Flags not supported: enable CONFIG_BOOTSTD_FULL\n");
 			return CMD_RET_USAGE;
 		boot = true;

More information about the U-Boot mailing list