Hi Al, On Fri, Nov 30 2012, Al Cooper wrote: > If "caps2" host capabilities does not indicate support for MMC > HS200, don't allow clock speeds >52MHz. Currently, for MMC, the > clock speed is set to the lesser of the max speed the eMMC module > supports (card->ext_csd.hs_max_dtr) or the max base clock of the > host controller (host->f_max based on BASE_CLK_FREQ in the host > CAPS register). This means that a host controller that doesn't > support HS200 mode but has a base clock of 100MHz and an eMMC module > that supports HS200 speeds will end up using a 100MHz clock. > > Signed-off-by: Al Cooper <alcooperx@xxxxxxxxx> > --- > drivers/mmc/core/mmc.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c > index 7cc4638..6d669c3 100644 > --- a/drivers/mmc/core/mmc.c > +++ b/drivers/mmc/core/mmc.c > @@ -1051,6 +1051,8 @@ static int mmc_init_card(struct mmc_host *host, u32 ocr, > if (mmc_card_highspeed(card) || mmc_card_hs200(card)) { > if (max_dtr > card->ext_csd.hs_max_dtr) > max_dtr = card->ext_csd.hs_max_dtr; > + if (mmc_card_highspeed(card) && (max_dtr > 52000000)) > + max_dtr = 52000000; > } else if (max_dtr > card->csd.max_dtr) { > max_dtr = card->csd.max_dtr; > } Thanks, pushed to mmc-next for 3.8. - Chris. -- Chris Ball <cjb@xxxxxxxxxx> <http://printf.net/> One Laptop Per Child -- 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