[PATCH 28/57] staging/rts_pstor: remove braces {} in sd.c (sd_tuning_tx)

2012-09-06 Thread Toshiaki Yamane 
fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane 
---
 drivers/staging/rts_pstor/sd.c |   24 ++--
 1 file changed, 10 insertions(+), 14 deletions(-)

diff --git a/drivers/staging/rts_pstor/sd.c b/drivers/staging/rts_pstor/sd.c
index 9812c00..02139b4 100644
--- a/drivers/staging/rts_pstor/sd.c
+++ b/drivers/staging/rts_pstor/sd.c
@@ -1779,17 +1779,16 @@ static int sd_tuning_tx(struct rtsx_chip *chip)
int (*tuning_cmd)(struct rtsx_chip *chip, u8 sample_point);
 
if (CHK_SD(sd_card)) {
-   if (CHK_SD_DDR50(sd_card)) {
+   if (CHK_SD_DDR50(sd_card))
tuning_cmd = sd_ddr_tuning_tx_cmd;
-   } else {
+   else
tuning_cmd = sd_sdr_tuning_tx_cmd;
-   }
+
} else {
-   if (CHK_MMC_DDR52(sd_card)) {
+   if (CHK_MMC_DDR52(sd_card))
tuning_cmd = sd_ddr_tuning_tx_cmd;
-   } else {
+   else
TRACE_RET(chip, STATUS_FAIL);
-   }
}
 
for (i = 0; i < 3; i++) {
@@ -1803,27 +1802,24 @@ static int sd_tuning_tx(struct rtsx_chip *chip)
}
 
retval = tuning_cmd(chip, (u8)j);
-   if (retval == STATUS_SUCCESS) {
+   if (retval == STATUS_SUCCESS)
raw_phase_map[i] |= 1 << j;
-   }
}
}
 
phase_map = raw_phase_map[0] & raw_phase_map[1] & raw_phase_map[2];
-   for (i = 0; i < 3; i++) {
+   for (i = 0; i < 3; i++)
RTSX_DEBUGP("TX raw_phase_map[%d] = 0x%08x\n", i, 
raw_phase_map[i]);
-   }
+
RTSX_DEBUGP("TX phase_map = 0x%08x\n", phase_map);
 
final_phase = sd_search_final_phase(chip, phase_map, TUNE_TX);
-   if (final_phase == 0xFF) {
+   if (final_phase == 0xFF)
TRACE_RET(chip, STATUS_FAIL);
-   }
 
retval = sd_change_phase(chip, final_phase, TUNE_TX);
-   if (retval != STATUS_SUCCESS) {
+   if (retval != STATUS_SUCCESS)
TRACE_RET(chip, STATUS_FAIL);
-   }
 
return STATUS_SUCCESS;
 }
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 28/57] staging/rts_pstor: remove braces {} in sd.c (sd_tuning_tx)

2012-09-06 Thread Toshiaki Yamane 
fixed below checkpatch warnings.
-WARNING: braces {} are not necessary for any arm of this statement
-WARNING: braces {} are not necessary for single statement blocks

Signed-off-by: Toshiaki Yamane yamaneto...@gmail.com
---
 drivers/staging/rts_pstor/sd.c |   24 ++--
 1 file changed, 10 insertions(+), 14 deletions(-)

diff --git a/drivers/staging/rts_pstor/sd.c b/drivers/staging/rts_pstor/sd.c
index 9812c00..02139b4 100644
--- a/drivers/staging/rts_pstor/sd.c
+++ b/drivers/staging/rts_pstor/sd.c
@@ -1779,17 +1779,16 @@ static int sd_tuning_tx(struct rtsx_chip *chip)
int (*tuning_cmd)(struct rtsx_chip *chip, u8 sample_point);
 
if (CHK_SD(sd_card)) {
-   if (CHK_SD_DDR50(sd_card)) {
+   if (CHK_SD_DDR50(sd_card))
tuning_cmd = sd_ddr_tuning_tx_cmd;
-   } else {
+   else
tuning_cmd = sd_sdr_tuning_tx_cmd;
-   }
+
} else {
-   if (CHK_MMC_DDR52(sd_card)) {
+   if (CHK_MMC_DDR52(sd_card))
tuning_cmd = sd_ddr_tuning_tx_cmd;
-   } else {
+   else
TRACE_RET(chip, STATUS_FAIL);
-   }
}
 
for (i = 0; i  3; i++) {
@@ -1803,27 +1802,24 @@ static int sd_tuning_tx(struct rtsx_chip *chip)
}
 
retval = tuning_cmd(chip, (u8)j);
-   if (retval == STATUS_SUCCESS) {
+   if (retval == STATUS_SUCCESS)
raw_phase_map[i] |= 1  j;
-   }
}
}
 
phase_map = raw_phase_map[0]  raw_phase_map[1]  raw_phase_map[2];
-   for (i = 0; i  3; i++) {
+   for (i = 0; i  3; i++)
RTSX_DEBUGP(TX raw_phase_map[%d] = 0x%08x\n, i, 
raw_phase_map[i]);
-   }
+
RTSX_DEBUGP(TX phase_map = 0x%08x\n, phase_map);
 
final_phase = sd_search_final_phase(chip, phase_map, TUNE_TX);
-   if (final_phase == 0xFF) {
+   if (final_phase == 0xFF)
TRACE_RET(chip, STATUS_FAIL);
-   }
 
retval = sd_change_phase(chip, final_phase, TUNE_TX);
-   if (retval != STATUS_SUCCESS) {
+   if (retval != STATUS_SUCCESS)
TRACE_RET(chip, STATUS_FAIL);
-   }
 
return STATUS_SUCCESS;
 }
-- 
1.7.9.5

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/