From: Marc Pignat <[EMAIL PROTECTED]>

MMC_POWER_ON is a noop, no need to set the power pin again.

Signed-off-by: Marc Pignat <[EMAIL PROTECTED]>
Signed-off-by: Nicolas Ferre <[EMAIL PROTECTED]>
---
MMC_POWER_OFF/MMC_POWER_UP turn the power OFF/ON, there is nothing to change
when ios->power_mode==MMC_POWER_ON.

Also changed indentation for the switch (CodingStyle), because there is no other
switch statement in this driver.

--- a/drivers/mmc/host/at91_mci.c
+++ b/drivers/mmc/host/at91_mci.c
@@ -658,13 +658,14 @@ static void at91_mci_set_ios(struct mmc_
        /* maybe switch power to the card */
        if (host->board->vcc_pin) {
                switch (ios->power_mode) {
-                       case MMC_POWER_OFF:
-                               at91_set_gpio_value(host->board->vcc_pin, 0);
-                               break;
-                       case MMC_POWER_UP:
-                       case MMC_POWER_ON:
-                               at91_set_gpio_value(host->board->vcc_pin, 1);
-                               break;
+               case MMC_POWER_OFF:
+                       at91_set_gpio_value(host->board->vcc_pin, 0);
+                       break;
+               case MMC_POWER_UP:
+                       at91_set_gpio_value(host->board->vcc_pin, 1);
+                       break;
+               default:
+                       break;
                }
        }
}


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

Reply via email to