Patchwork [U-Boot] vision2: Fix checkpatch warning

login
register
mail settings
Submitter Fabio Estevam
Date Nov. 21, 2011, 3:57 p.m.
Message ID <1321891059-15087-1-git-send-email-fabio.estevam@freescale.com>
Download mbox | patch
Permalink /patch/126822/
State Accepted
Commit afffcc7bfa7634ecd733788d7f4d662c43e09727
Delegated to: Albert ARIBAUD
Headers show

Comments

Fabio Estevam - Nov. 21, 2011, 3:57 p.m.
Fix the following checkpatch warning:

WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 board/ttcontrol/vision2/vision2.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)
Albert ARIBAUD - Nov. 28, 2011, 12:29 p.m.
Hi Fabio,

Le 21/11/2011 16:57, Fabio Estevam a écrit :
> Fix the following checkpatch warning:
>
> WARNING: braces {} are not necessary for single statement blocks
>
> Signed-off-by: Fabio Estevam<fabio.estevam@freescale.com>
> ---
>   board/ttcontrol/vision2/vision2.c |    3 +--
>   1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/board/ttcontrol/vision2/vision2.c b/board/ttcontrol/vision2/vision2.c
> index f556d30..282de95 100644
> --- a/board/ttcontrol/vision2/vision2.c
> +++ b/board/ttcontrol/vision2/vision2.c
> @@ -428,9 +428,8 @@ static void setup_gpios(void)
>   	gpio_direction_output(4, 1);
>
>   	gpio_direction_output(7, 0);
> -	for (i = 65; i<  71; i++) {
> +	for (i = 65; i<  71; i++)
>   		gpio_direction_output(i, 0);
> -	}
>
>   	gpio_direction_output(94, 0);

(chose to add to next as it is not a bugfix per se; vision2 builds fine 
already without this patch)

Added to u-boot-arm/next, thanks!

Amicalement,

Patch

diff --git a/board/ttcontrol/vision2/vision2.c b/board/ttcontrol/vision2/vision2.c
index f556d30..282de95 100644
--- a/board/ttcontrol/vision2/vision2.c
+++ b/board/ttcontrol/vision2/vision2.c
@@ -428,9 +428,8 @@  static void setup_gpios(void)
 	gpio_direction_output(4, 1);
 
 	gpio_direction_output(7, 0);
-	for (i = 65; i < 71; i++) {
+	for (i = 65; i < 71; i++)
 		gpio_direction_output(i, 0);
-	}
 
 	gpio_direction_output(94, 0);