On Fri, 29 Jul 2016 21:17:30 +0200 Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> wrote: > > > What happens if you actually want to set it to 100MHz? > > > > There is no SDXC_CLK_100M in the mainline driver, and 100MHz is asked > > only for 8 bits DDR at 50MHz. > > You're missing the point. > > clk_set_rate is supposed to apply a rate as close as possible as > requested, there's no reason why you would request a rate twice as > high as need. > > You want to switch the clock from one mode to another, fine, create a > new function for that. But don't hack an existing one. I will not change the core clock stuff for this marginal case. Flagging the clock as 'new mode capable' is enough. Anyway, setting the 'new mode' bit to the clock divides the rate by two, so, there is no reason to not use it. -- Ken ar c'hentañ | ** Breizh ha Linux atav! ** Jef | http://moinejf.free.fr/ -- 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
