Hi Jerry,

On 12/6/2011 5:18 PM, [email protected] wrote:
> From: Jerry Huang <[email protected]>
>
> When f_init is zero, the SDHC can't work correctly. So f_min will replace
> f_init, when f_init is zero.
>
> Signed-off-by: Jerry Huang <[email protected]>
> CC: Chris Ball <[email protected]>
> ---
> changes for v2:
>       - add the CC
>
>  drivers/mmc/core/core.c |    5 ++++-
>  1 files changed, 4 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c
> index a08e6b1..2d40c04 100644
> --- a/drivers/mmc/core/core.c
> +++ b/drivers/mmc/core/core.c
> @@ -1253,7 +1253,10 @@ static void mmc_power_up(struct mmc_host *host)
>        */
>       mmc_delay(10);
>  
> -     host->ios.clock = host->f_init;
> +     if (host->f_init)
> +             host->ios.clock = host->f_init;
> +     else
> +             host->ios.clock = host->f_min;

Are you sure f_min can have a value of zero?
It should have been set mmc_rescan_try_freq(), when trying a list
of frequencies: 400000, 300000, 200000, 100000

>  
>       host->ios.power_mode = MMC_POWER_ON;
>       mmc_set_ios(host);
--
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

Reply via email to