[PATCH] staging: rts5208: ensure braces on all arms of if stmt

From	Nicholas Sim <nicholassimws@gmail.com>
Newsgroups	linux.kernel
Subject	[PATCH] staging: rts5208: ensure braces on all arms of if stmt
Date	2016-04-05 18:30 +0200
Message-ID	<rkC1C-6PL-51@gated-at.bofh.it> (permalink)
Organization	linux.* mail to news gateway

Show all headers | View raw

Added braces on if arm of if statement where else arm already needs braces
as suggested for clarity in Documentation/CodingStyle

Signed-off-by: Nicholas Sim <nicholassimws@gmail.com>
---
 drivers/staging/rts5208/ms.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/rts5208/ms.c b/drivers/staging/rts5208/ms.c
index 3e75db7..2ad4ea0 100644
--- a/drivers/staging/rts5208/ms.c
+++ b/drivers/staging/rts5208/ms.c
@@ -794,8 +794,9 @@ static int ms_confirm_cpu_startup(struct rtsx_chip *chip)
 	}
 
 	if (val & INT_REG_ERR) {
-		if (val & INT_REG_CMDNK)
+		if (val & INT_REG_CMDNK) {
 			chip->card_wp |= (MS_CARD);
+		}
 		else {
 			rtsx_trace(chip);
 			return STATUS_FAIL;
-- 
2.4.3

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

[PATCH] staging: rts5208: ensure braces on all arms of if stmt Nicholas Sim <nicholassimws@gmail.com> - 2016-04-05 18:30 +0200
  Re: [PATCH] staging: rts5208: ensure braces on all arms of if stmt Dan Carpenter <dan.carpenter@oracle.com> - 2016-04-06 14:40 +0200

csiph-web