On 16 January 2017 at 17:56, Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> wrote: > The MMC core assumes that the code will gate the clock when the bus > frequency is set to 0, which we've been ignoring so far. > > Handle that. > > Signed-off-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> Looks good! Kind regards Uffe > --- > drivers/mmc/host/sunxi-mmc.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/mmc/host/sunxi-mmc.c b/drivers/mmc/host/sunxi-mmc.c > index ab4324e6eb74..019f95e8e7c5 100644 > --- a/drivers/mmc/host/sunxi-mmc.c > +++ b/drivers/mmc/host/sunxi-mmc.c > @@ -765,6 +765,9 @@ static int sunxi_mmc_clk_set_rate(struct sunxi_mmc_host *host, > if (ret) > return ret; > > + if (!ios->clock) > + return 0; > + > /* 8 bit DDR requires a higher module clock */ > if (ios->timing == MMC_TIMING_MMC_DDR52 && > ios->bus_width == MMC_BUS_WIDTH_8) > @@ -882,7 +885,7 @@ static void sunxi_mmc_set_ios(struct mmc_host *mmc, struct mmc_ios *ios) > mmc_writel(host, REG_GCTRL, rval); > > /* set up clock */ > - if (ios->clock && ios->power_mode) { > + if (ios->power_mode) { > host->ferror = sunxi_mmc_clk_set_rate(host, ios); > /* Android code had a usleep_range(50000, 55000); here */ > } > -- > git-series 0.8.11 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html