mmc: core: Skip to set the default 200mA SD current limit

Let's avoid updating the SD current limit when the maximum power is 200mA
(0.72W) or less, as this is already the default value for the SD card. In
this way we avoid sending an unnecessary command during initialization.

Signed-off-by: Avri Altman <avri.altman@sandisk.com>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>

diff --git a/drivers/mmc/core/sd.c b/drivers/mmc/core/sd.c
index b6758911e22cda08d22800b6772b4d4c8438561d..948948ca9b4ae1598d375d4056fb635f4dda0911 100644 (file)
--- a/drivers/mmc/core/sd.c
+++ b/drivers/mmc/core/sd.c
@@ -554,7 +554,7 @@ static u32 sd_get_host_max_current(struct mmc_host *host)
 
 static int sd_set_current_limit(struct mmc_card *card, u8 *status)
 {
-       int current_limit = SD_SET_CURRENT_NO_CHANGE;
+       int current_limit = SD_SET_CURRENT_LIMIT_200;
        int err;
        u32 max_current;
 
@@ -598,11 +598,8 @@ static int sd_set_current_limit(struct mmc_card *card, u8 *status)
        else if (max_current >= 400 &&
                 card->sw_caps.sd3_curr_limit & SD_MAX_CURRENT_400)
                current_limit = SD_SET_CURRENT_LIMIT_400;
-       else if (max_current >= 200 &&
-                card->sw_caps.sd3_curr_limit & SD_MAX_CURRENT_200)
-               current_limit = SD_SET_CURRENT_LIMIT_200;
 
-       if (current_limit != SD_SET_CURRENT_NO_CHANGE) {
+       if (current_limit != SD_SET_CURRENT_LIMIT_200) {
                err = mmc_sd_switch(card, SD_SWITCH_SET, 3,
                                current_limit, status);
                if (err)

diff --git a/include/linux/mmc/card.h b/include/linux/mmc/card.h
index ddcdf23d731c4732402920b82fe48cc019158ac3..e9e964c20e5307fa91000c1af34dda36faac0ff5 100644 (file)
--- a/include/linux/mmc/card.h
+++ b/include/linux/mmc/card.h
@@ -182,7 +182,6 @@ struct sd_switch_caps {
 #define SD_SET_CURRENT_LIMIT_400       1
 #define SD_SET_CURRENT_LIMIT_600       2
 #define SD_SET_CURRENT_LIMIT_800       3
-#define SD_SET_CURRENT_NO_CHANGE       (-1)
 
 #define SD_MAX_CURRENT_200     (1 << SD_SET_CURRENT_LIMIT_200)
 #define SD_MAX_CURRENT_400     (1 << SD_SET_CURRENT_LIMIT_400)