diff options
| author | Quentin Schulz <[email protected]> | 2025-11-12 18:48:06 +0100 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2025-12-03 11:03:45 -0600 |
| commit | 29a7fe55c557e462457f4fb3dcb25681952b0803 (patch) | |
| tree | e95b6563c45a226cbd0d49b4fdb0e4802b71744c /cmd/led.c | |
| parent | caa740df9cae5ddc74b18b88a4fea43e27b2fdf6 (diff) | |
cmd: led: fix help text for blink subcommand
The blink subcommand actually requires an additional parameter
(blink-freq) but not the others.
In order to simplify the help text, split the blink subcommand help text
from the off|on|toggle subcommands.
Then, fix the help text so that it is clear that the frequency is
required.
While at it, specify the duty cycle.
Fixes: ffe2052d6e8a ("dm: led: Add a new 'led' command")
Signed-off-by: Quentin Schulz <[email protected]>
Diffstat (limited to 'cmd/led.c')
| -rw-r--r-- | cmd/led.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/cmd/led.c b/cmd/led.c index 91fb856ee59..296c07b3b38 100644 --- a/cmd/led.c +++ b/cmd/led.c @@ -118,16 +118,13 @@ int do_led(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[]) return 0; } -#if defined(CONFIG_LED_BLINK) || defined(CONFIG_LED_SW_BLINK) -#define BLINK "|blink [blink-freq in ms]" -#else -#define BLINK "" -#endif - U_BOOT_CMD( led, 4, 1, do_led, "manage LEDs", - "<led_label> on|off|toggle" BLINK "\tChange LED state\n" + "<led_label> on|off|toggle\tChange LED state\n" +#if defined(CONFIG_LED_BLINK) || defined(CONFIG_LED_SW_BLINK) + "led <led_label> blink <blink-freq in ms>\tBlink LED (duty cycle 50%)\n" +#endif "led <led_label>\tGet LED state\n" "led list\t\tshow a list of LEDs" ); |
