On 01/10/2014 09:39 PM, Seungwon Jeon wrote: >> +static void dw_mci_k3_set_ios(struct dw_mci *host, struct mmc_ios *ios) >> +{ >> + struct dw_mci_k3_priv_data *priv = host->priv; >> + u32 rate = priv->clk_table[ios->timing]; > > First, sorry for quick review even though your effort. > But I still worry about this change. > Currently k3 host's clock table depends on value number of SD/MMC mode value. > It seems close to identifier for eachg mode. I think it's not good way to use as table's index. > Can you modify to mmc_clk_determine_rate() in your another patch-set? > I guess required actual target rate could be determined depending on ios->clock. > No, it can not get input clock source rate simply from ios->clock, also requied info like which controller, which mode etc. Here is setting clock source rate, not the working clock rate. For example, emmc init clock source rate is 13M, while sd init clock source is 25M, it can not simply get such info from ios->clock. And for HS200, emmc may have to set clock source rate to 104M since the controller limitation and can not work stable as 208M. Thanks -- 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