On 1 September 2014 14:20, Robert Jarzmik <robert.jarzmik@xxxxxxx> wrote: > Ulf Hansson <ulf.hansson@xxxxxxxxxx> writes: > >> >> I would suggest you to re-place the existing clk_enable() with >> clk_prepare_enable() and clk_disable with clk_disable_unprepare() >> instead of the approach taken in this patch. > > I don't think it works in this driver. > > Look at pxamci_set_ios(), and all the handling of host->clkrt (CLKRT_OFF, > ...). This driver enables the clock only when transfers are required, not all > the time. Therefore I'd rather let it be that way. I understand your concern, but I don't see why there should be any major difference in clock management code (clk tree wise), due to this patch. It worked before, so likely it will work now!? Kind regards Uffe -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
