sandbox: Update do_host_bind() argument counting
authorSimon Glass <sjg@chromium.org>
Mon, 15 Mar 2021 05:11:10 +0000 (18:11 +1300)
committerSimon Glass <sjg@chromium.org>
Sat, 27 Mar 2021 02:04:31 +0000 (15:04 +1300)
Remove the 'bind' subcommand before processing the arguments. This will
make it easier to add an optional flag.

Signed-off-by: Simon Glass <sjg@chromium.org>
cmd/host.c

index 927c23d..847bb1d 100644 (file)
@@ -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);
 }