: staging: Greybus: Remove unnecessary braces for single statement block

PATCH Superseded Rahul Krishnan 2 participants 2 messages Nov 15, 2016

Subsystems (3)

Staging Greybus Greybus SDIO/GPIO/SPI protocols drivers

lore

Share & tools

lore

Rahul Krishnan

author

2016-11-15 13:25 UTC #0 lore raw

PATCH 1/1 : staging: Greybus: Remove unnecessary braces for single statement block

This patch fixes the following checkpath.pl warning
WARNING: braces {} are not necessary for single statement blocks

Signed-off-by Rahul Krishnan <[email protected]>

---
 drivers/staging/greybus/sdio.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

drivers/staging/greybus/sdio.c +2 -3

index 5649ef1..f60b458 100644
@@ -191,9 +191,8 @@ static int _gb_sdio_process_events(struct gb_sdio_host *host, u8 event)
 		state_changed = 1;
 	}
 
-	if (event & GB_SDIO_WP) {
-		host->read_only = true;
-	}
+	if (event & GB_SDIO_WP)
+	host->read_only = true;
 
 	if (state_changed) {
 		dev_info(mmc_dev(host->mmc), "card %s now event\n",

Signature · 1 line

2.10.2

Dan Carpenter

2016-11-15 13:27 UTC #1 lore raw

Quoted 13 lines · Rahul Krishnan

This patch fixes the following checkpath.pl warning
WARNING: braces {} are not necessary for single statement blocks
Signed-off-by Rahul Krishnan <[email protected]>
---
 drivers/staging/greybus/sdio.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
drivers/staging/greybus/sdio.c +0 -0
index 5649ef1..f60b458 100644

drivers/staging/greybus/sdio.c static int _gb_sdio_process_events(struct gb_sdio_host *host, u8 event) · line 195

Quoted 8 lines

drivers/staging/greybus/sdio.c +2 -3

@@ -191,9 +191,8 @@ static int _gb_sdio_process_events(struct gb_sdio_host *host, u8 event)
 		state_changed = 1;
 	}
 
-	if (event & GB_SDIO_WP) {
-		host->read_only = true;
-	}
+	if (event & GB_SDIO_WP)
+	host->read_only = true;

The indenting is messed up.

regards,
dan carpenter