[U-Boot] [PATCH] cmd: led.c: fix coding style

Tom Rini trini at konsulko.com
Mon Aug 13 16:50:37 UTC 2018


On Sun, Aug 12, 2018 at 03:06:03PM +0800, Akee Huang wrote:

> Fix coding style according to checkpatch.pl
> 
> Signed-off-by: Akee Huang <akeeoao at gmail.com>
> ---
>  cmd/led.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/cmd/led.c b/cmd/led.c
> index fc07ca95a3..b339c98dfb 100644
> --- a/cmd/led.c
> +++ b/cmd/led.c
> @@ -133,10 +133,10 @@ int do_led(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
>  #define BLINK ""
>  #endif
>  
> -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 list\t\tshow a list of LEDs"
> +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 list\t\tshow a list of LEDs"
>  );

This is indeed a style violation.  However, a quick 'git grep' shows
that we really don't have consistency here.  And to the extent that we
do it's following the style that checkpatch.pl doesn't like and we're
doing in cmd/led.c.  So, in the absence of a Coccinelle (or simlar) script
and proposal to get everything consistent, I don't think this patch by
itself is a good idea to apply, sorry.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180813/e1b6988e/attachment.sig>


More information about the U-Boot mailing list