[...] >>> + if (clk_rate != clk_get_rate(host->cfg_div_clk)) >>> + dev_dbg(host->dev, >>> + "divider requested rate %lu != actual rate %lu\n", >>> + clk_rate, clk_get_rate(host->cfg_div_clk)); >>> + >>> + mmc->actual_clock = clk_rate; >> >> debug message is described actual_rate as clk_get_rate().. >> What is correct "actual_clock"? >> > Indeed this might be a little misleading. > "actual_clock" is a member of struct mmc_host and for now we have to live > with this name. However the meaning in our context is: "currently requested > clock rate". I think this is wrong. The intent with mmc->actual_clock is to show what the real rate of the clock is, not the requested rate. The requested clock rate is already available via "mmc->ios.clock". Hopes this clarifies it. [...] 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
