On Mon, Sep 20, 2010 at 10:34 PM, David Vrabel <[email protected]> wrote: > zhangfei gao wrote: >> From fa357144df43e6c1a9e5e36af497c53437cd41e1 Mon Sep 17 00:00:00 2001 >> From: Zhangfei Gao <[email protected]> >> Date: Mon, 20 Sep 2010 15:12:26 -0400 >> Subject: [PATCH 3/4] mmc: fix init f_min >> >> Change-Id: I7fab1a1a162d9f26fc92c497078f26912c04a3fc > > What's this line in the description for? > >> Signed-off-by: Zhangfei Gao <[email protected]> >> --- >> drivers/mmc/core/core.c | 4 ++-- >> 1 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c >> index 5db49b1..9114c87 100644 >> --- a/drivers/mmc/core/core.c >> +++ b/drivers/mmc/core/core.c >> @@ -910,9 +910,9 @@ static void mmc_power_up(struct mmc_host *host) >> if (host->f_min > 400000) { >> pr_warning("%s: Minimum clock frequency too high for " >> "identification mode\n", mmc_hostname(host)); >> - host->ios.clock = host->f_min; >> - } else >> host->ios.clock = 400000; >> + } else >> + host->ios.clock = host->f_min; > > NAK. > > The code is already correctly requesting 400 kHz (unless the controller > can't go that slow). >
Maybe i misunderstood. Is it mean using f_min when f_min > 400000, for example, when f_min = 2000000, use f_min? But using 400000 when f_min is smaller than 400000, then how about the card not support 400000 at init. > David > -- > David Vrabel, Senior Software Engineer, Drivers > CSR, Churchill House, Cambridge Business Park, Tel: +44 (0)1223 692562 > Cowley Road, Cambridge, CB4 0WZ http://www.csr.com/ > > > Member of the CSR plc group of companies. CSR plc registered in England and > Wales, registered number 4187346, registered office Churchill House, > Cambridge Business Park, Cowley Road, Cambridge, CB4 0WZ, United Kingdom > -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
