mmc: dw_mmc: rockchip: Don't recalc the clock when it goes off
authorDoug Anderson <dianders@chromium.org>
Tue, 14 Oct 2014 16:39:12 +0000 (09:39 -0700)
committerUlf Hansson <ulf.hansson@linaro.org>
Mon, 10 Nov 2014 11:40:36 +0000 (12:40 +0100)
The "set_ios" function is called with a clock of 0 when the clock is
turning off.  There's no reason to go through all the extra Rockchip
logic (whose goal is to make sure DIV is 0 or 1) in that case.  The
Rockchip logic happened to work because the CCF will pick the lowest
possible rate when you ask it for a clock of 0, but it's silly to go
through all the remuxing and adjusting for no reason.

Signed-off-by: Doug Anderson <dianders@chromium.org>
Acked-by: Jaehoon Chung <jh80.chung@samsung.com>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
drivers/mmc/host/dw_mmc-rockchip.c

index f0c2cb1a210d73ad284584b938fe044412cda4e4..bbb4ec386e56aea1f0bac641b0b1e8bb9e86c35f 100644 (file)
@@ -37,6 +37,9 @@ static void dw_mci_rk3288_set_ios(struct dw_mci *host, struct mmc_ios *ios)
        unsigned int cclkin;
        u32 bus_hz;
 
+       if (ios->clock == 0)
+               return;
+
        /*
         * cclkin: source clock of mmc controller
         * bus_hz: card interface clock generated by CLKGEN