Patchwork [U-Boot] cmd_led.c: Standardize format of help and usage info.

login
register
mail settings
Submitter Robert P. J. Day
Date Nov. 11, 2012, 8:39 p.m.
Message ID <alpine.DEB.2.02.1211111536430.885@oneiric>
Download mbox | patch
Permalink /patch/198299/
State Accepted
Delegated to: Tom Rini
Headers show

Comments

Robert P. J. Day - Nov. 11, 2012, 8:39 p.m.
Current "led" help and usage info has redundancy and extraneous
newlines, tweak it to be consistent with other commands.

Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>

---

  checked custodians page, didn't notice anyone in charge of LEDs.
Tom Rini - Dec. 7, 2012, 3:50 p.m.
On Sun, Nov 11, 2012 at 10:39:07AM -0000, Robert P. J. Day wrote:

> Current "led" help and usage info has redundancy and extraneous
> newlines, tweak it to be consistent with other commands.
> 
> Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>

Applied to u-boot/master, thanks!

Patch

diff --git a/common/cmd_led.c b/common/cmd_led.c
index d83b3ba..7f5ab43 100644
--- a/common/cmd_led.c
+++ b/common/cmd_led.c
@@ -140,7 +140,7 @@  int do_led (cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])

 U_BOOT_CMD(
 	led, 3, 1, do_led,
-	"led\t- ["
+	"["
 #ifdef CONFIG_BOARD_SPECIFIC_LED
 #ifdef STATUS_LED_BIT
 	"0|"
@@ -167,6 +167,6 @@  U_BOOT_CMD(
 #ifdef STATUS_LED_BLUE
 	"blue|"
 #endif
-	"all] [on|off|toggle]\n",
-	"led [led_name] [on|off|toggle] sets or clears led(s)\n"
+	"all] [on|off|toggle]",
+	"[led_name] [on|off|toggle] sets or clears led(s)"
 );