Staging: rts5208: Use min instead of ternary operator

This patch replaces ternary operator with macro min as it shorter and
thus increases code readability. Macro min returns the minimum of the
two compared values.
Made a semantic patch for changes:

@@
type T;
T x;
T y;
@@
(
- x < y ? x : y
+ min(x,y)
|
- x > y ? x : y
+ max(x,y)
)

Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/rts5208/sd.c b/drivers/staging/rts5208/sd.c
index 244d589db35998bd41e2e69f4ebd7347c084c24e..1612610c1f565d1fcf0d6a01b2f4486e40da8327 100644 (file)
--- a/drivers/staging/rts5208/sd.c
+++ b/drivers/staging/rts5208/sd.c
@@ -303,7 +303,7 @@ static int sd_read_data(struct rtsx_chip *chip,
 
        if (cmd_len) {
                dev_dbg(rtsx_dev(chip), "SD/MMC CMD %d\n", cmd[0] - 0x40);
-               for (i = 0; i < (cmd_len < 6 ? cmd_len : 6); i++)
+               for (i = 0; i < (min(cmd_len, 6)); i++)
                        rtsx_add_cmd(chip, WRITE_REG_CMD, REG_SD_CMD0 + i,
                                     0xFF, cmd[i]);
        }
@@ -383,7 +383,7 @@ static int sd_write_data(struct rtsx_chip *chip, u8 trans_mode,
 
        if (cmd_len) {
                dev_dbg(rtsx_dev(chip), "SD/MMC CMD %d\n", cmd[0] - 0x40);
-               for (i = 0; i < (cmd_len < 6 ? cmd_len : 6); i++) {
+               for (i = 0; i < (min(cmd_len, 6)); i++) {
                        rtsx_add_cmd(chip, WRITE_REG_CMD,
                                     REG_SD_CMD0 + i, 0xFF, cmd[i]);
                }