[PATCH 05/19] sandbox: Update do_host_bind() argument counting

Simon Glass sjg at chromium.org
Mon Mar 15 06:11:10 CET 2021


Remove the 'bind' subcommand before processing the arguments. This will
make it easier to add an optional flag.

Signed-off-by: Simon Glass <sjg at chromium.org>
---

 cmd/host.c | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/cmd/host.c b/cmd/host.c
index 927c23d0d9d..847bb1d3b5f 100644
--- a/cmd/host.c
+++ b/cmd/host.c
@@ -46,15 +46,18 @@ static int do_host_bind(struct cmd_tbl *cmdtp, int flag, int argc,
 	char *ep;
 	int dev;
 
-	if (argc < 2 || argc > 3)
+	/* Skip 'bind' */
+	argc--;
+	argv++;
+	if (argc < 1 || argv > 2)
 		return CMD_RET_USAGE;
-	dev_str = argv[1];
+	dev_str = argv[0];
 	dev = simple_strtoul(dev_str, &ep, 16);
 	if (*ep) {
 		printf("** Bad device specification %s **\n", dev_str);
 		return CMD_RET_USAGE;
 	}
-	file = argc >= 3 ? argv[2] : NULL;
+	file = argc > 1 ? argv[1] : NULL;
 
 	return !!host_dev_bind(dev, file);
 }
-- 
2.31.0.rc2.261.g7f71774620-goog



More information about the U-Boot mailing list