: staging: Greybus: Remove unnecessary braces for single statement block
Subsystems (3)
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 | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
drivers/staging/greybus/sdio.c +1 -2
@@ -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 ) { +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
Hi Rahul,
Thanks for the patch.
Now looks great.
Quoted 5 lines · Rahul Krishnan
This patch fixes the following checkpath.pl warning
WARNING: braces {} are not necessary for single statement blocksSigned-off-by Rahul Krishnan <[email protected]>
Reviewed-by Rui Miguel Silva <[email protected]>
Quoted 22 lines
---
drivers/staging/greybus/sdio.c| 3 +--1 file changed, 1 insertion(+), 2 deletions(-)drivers/staging/greybus/sdio.c +1
-2@@ -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 ) { +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
Quoted 8 lines · Rahul Krishnan
This patch fixes the following checkpath.pl warning
WARNING: braces {} are not necessary for single statement blocksSigned-off-by Rahul Krishnan <[email protected]>--- drivers/staging/greybus/sdio.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
Acked-by Viresh Kumar <[email protected]>