On Wed, Oct 13, 2010 at 11:07 AM, Adrian Hunter <[email protected]> wrote:
>> +static int mmc_clock_opt_set(void *data, u64 val)
>> +{
>> +       struct mmc_host *host = data;
>> +
>> +       /* We need this check due to input value is u64 */
>> +       if (val>  host->f_max)
>> +               return -EINVAL;
>> +
>> +       mmc_set_clock(host, (unsigned int) val);
>
> Probably need to claim the host here otherwise you might
> be changing the clock while a request is in progress
Yeah, good catch. Will fix. Thanks.

-- 
With Best Regards,
Andy Shevchenko
--
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