Skip to contentSkip to conversation

rts5208:Fix checkpatch warnings

PATCH Superseded Ravi Teja Darbha 2 participants 2 messages Jun 30, 2015 Staging

lore
Share & tools
lore

Ravi Teja Darbha

author
#0 lore raw

PATCH 1/1 rts5208:Fix checkpatch warnings

Else condition after if statement is not necessary, hence removed.

Signed-off-by Ravi Teja Darbha <[email protected]>
---
 drivers/staging/rts5208/xd.c | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)
drivers/staging/rts5208/xd.c +4 -6
index 10fea7b..42d8d82 100644
@@ -1689,10 +1689,9 @@ static int xd_read_multiple_pages(struct rtsx_chip *chip, u32 phy_blk,
 			xd_set_err_code(chip, XD_TO_ERROR);
 			rtsx_trace(chip);
 			return STATUS_FAIL;
-		} else {
-			rtsx_trace(chip);
-			goto Fail;
 		}
+		rtsx_trace(chip);
+		goto Fail;
 	}
 
 	return STATUS_SUCCESS;

@@ -1905,10 +1904,9 @@ static int xd_write_multiple_pages(struct rtsx_chip *chip, u32 old_blk,
 			xd_set_err_code(chip, XD_TO_ERROR);
 			rtsx_trace(chip);
 			return STATUS_FAIL;
-		} else {
-			rtsx_trace(chip);
-			goto Fail;
 		}
+		rtsx_trace(chip);
+		goto Fail;
 	}
 
 	if (end_page == (xd_card->page_off + 1)) {
Signature · 1 line
1.9.1
#1 lore raw
Quoted 1 line · Ravi Teja Darbha

Else condition after if statement is not necessary, hence removed.

else is not necessary after if??? then what it is supposed to be used? :)

I think your commit message should have been:
"else is not necessary after return statement."

regards
sudip