The power-mode sequence on MMC is MMC_POWER_OFF -> MMC_POWER_UP ->
MMC_POWER_ON and not MMC_POWER_ON -> MMC_POWER_UP, as the driver currently
is implying.

Signed-off-by: Guennadi Liakhovetski <[email protected]>
---

Please, test on MFD!

V2: rebase on top of driver split

 drivers/mmc/host/tmio_mmc_pio.c |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/mmc/host/tmio_mmc_pio.c b/drivers/mmc/host/tmio_mmc_pio.c
index e1aed25..69772ab 100644
--- a/drivers/mmc/host/tmio_mmc_pio.c
+++ b/drivers/mmc/host/tmio_mmc_pio.c
@@ -746,20 +746,20 @@ static void tmio_mmc_set_ios(struct mmc_host *mmc, struct 
mmc_ios *ios)
        if (ios->clock)
                tmio_mmc_set_clock(host, ios->clock);
 
-       /* Power sequence - OFF -> ON -> UP */
+       /* Power sequence - OFF -> UP -> ON */
        switch (ios->power_mode) {
        case MMC_POWER_OFF: /* power down SD bus */
                if (host->set_pwr)
                        host->set_pwr(host->pdev, 0);
                tmio_mmc_clk_stop(host);
                break;
-       case MMC_POWER_ON: /* power up SD bus */
+       case MMC_POWER_ON: /* start bus clock */
+               tmio_mmc_clk_start(host);
+               break;
+       case MMC_POWER_UP: /* power up SD bus */
                if (host->set_pwr)
                        host->set_pwr(host->pdev, 1);
                break;
-       case MMC_POWER_UP: /* start bus clock */
-               tmio_mmc_clk_start(host);
-               break;
        }
 
        switch (ios->bus_width) {
-- 
1.7.2.3

--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to