cmd/led: check subcommand "list" instead "l"
authorHeiko Schocher <hs@denx.de>
Mon, 27 May 2019 06:14:16 +0000 (08:14 +0200)
committerTom Rini <trini@konsulko.com>
Wed, 5 Jun 2019 16:14:02 +0000 (12:14 -0400)
current implementation for checking if "led list"
command is called checks only if "l" is passed to the
led command. This prevents switching leds with name
which starts also with a "l". So check for passing
"list".

While at it, also fix a typo in led command usage.

Signed-off-by: Heiko Schocher <hs@denx.de>
cmd/led.c

index fc07ca95a3153e36faa5d3ff7f385440789880f1..403abbc6bcf2666f96b0a28ee2a785d8dfda8545 100644 (file)
--- a/cmd/led.c
+++ b/cmd/led.c
@@ -85,7 +85,7 @@ int do_led(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
        if (argc < 2)
                return CMD_RET_USAGE;
        led_label = argv[1];
-       if (*led_label == 'l')
+       if (strncmp(led_label, "list", 4) == 0)
                return list_leds();
 
        cmd = argc > 2 ? get_led_cmd(argv[2]) : LEDST_COUNT;
@@ -137,6 +137,6 @@ U_BOOT_CMD(
        led, 4, 1, do_led,
        "manage LEDs",
        "<led_label> on|off|toggle" BLINK "\tChange LED state\n"
-       "led [<led_label>\tGet LED state\n"
+       "led [<led_label>]\tGet LED state\n"
        "led list\t\tshow a list of LEDs"
 );