Dear Chris,

I will change the comment, and re-send the patch.
Thanks Chris.

Best Regards,
Jaehoon Chung

On 06/27/2013 11:53 PM, Chris Ball wrote:
> Hi Jaehoon,
> 
> On Mon, May 27 2013, Jaehoon Chung wrote:
>> When use the QUIRK_NONSTANDARD_CLOCK, then never set to 0 at clock control 
>> register.
>> This patch is fixed this problem.
>>
>> Signed-off-by: Jaehoon Chung <[email protected]
>> Signed-off-by: Kyungmin Park <[email protected]>
>> ---
>> Changelog v2:
>> - Rebased at the latest mmc tree
>>
>>  drivers/mmc/host/sdhci-s3c.c |    5 ++++-
>>  1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/mmc/host/sdhci-s3c.c b/drivers/mmc/host/sdhci-s3c.c
>> index 926aaf6..ce71f32 100644
>> --- a/drivers/mmc/host/sdhci-s3c.c
>> +++ b/drivers/mmc/host/sdhci-s3c.c
>> @@ -297,8 +297,11 @@ static void sdhci_cmu_set_clock(struct sdhci_host 
>> *host, unsigned int clock)
>>      u16 clk = 0;
>>  
>>      /* don't bother if the clock is going off */
>> -    if (clock == 0)
>> +    if (clock == 0) {
>> +            sdhci_writew(host, 0, SDHCI_CLOCK_CONTROL);
>> +            host->clock = clock;
>>              return;
>> +    }
>>  
>>      sdhci_s3c_set_clock(host, clock);
> 
> You should change the comment too, since your patch invalidates it.
> 
> Thanks,
> 
> - Chris.
> 

--
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