mirror of
https://github.com/AsahiLinux/u-boot
synced 2024-11-10 23:24:38 +00:00
cmd/led: check subcommand "list" instead "l"
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>
This commit is contained in:
parent
c5646270d1
commit
ea41b15617
1 changed files with 2 additions and 2 deletions
|
@ -85,7 +85,7 @@ int do_led(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
|
||||||
if (argc < 2)
|
if (argc < 2)
|
||||||
return CMD_RET_USAGE;
|
return CMD_RET_USAGE;
|
||||||
led_label = argv[1];
|
led_label = argv[1];
|
||||||
if (*led_label == 'l')
|
if (strncmp(led_label, "list", 4) == 0)
|
||||||
return list_leds();
|
return list_leds();
|
||||||
|
|
||||||
cmd = argc > 2 ? get_led_cmd(argv[2]) : LEDST_COUNT;
|
cmd = argc > 2 ? get_led_cmd(argv[2]) : LEDST_COUNT;
|
||||||
|
@ -137,6 +137,6 @@ U_BOOT_CMD(
|
||||||
led, 4, 1, do_led,
|
led, 4, 1, do_led,
|
||||||
"manage LEDs",
|
"manage LEDs",
|
||||||
"<led_label> on|off|toggle" BLINK "\tChange LED state\n"
|
"<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"
|
"led list\t\tshow a list of LEDs"
|
||||||
);
|
);
|
||||||
|
|
Loading…
Reference in a new issue