Re: [PATCH v2] mmc_rescan: reduce verbosity

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



 On 12-10-2010 23:08, Chris Ball wrote:
> Hi Eric,
>
> On Tue, Oct 12, 2010 at 09:29:37AM +0200, Eric Bénard wrote:
>> since commit 820f2bcfdc32cd90061224c930cf670f961e12d1 mmc_rescan
>> includes a pr_info which prints 4 lines each second for hosts configured
>> with MMC_CAP_NEEDS_POLL. Tested on i.MX51's sdhci-esdhc.
>> This patch enable the log only if CONFIG_MMC_DEBUG is selected.
>>
>> Signed-off-by: Eric Bénard <eric@xxxxxxxxxx>
>> ---
>> v2 : enable log only if CONFIG_MMC_DEBUG is selected
>> v1 : use pr_debug instead of pr_info
>>  drivers/mmc/core/core.c |    3 ++-
>>  1 files changed, 2 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c
>> index c94565d..cda9d29 100644
>> --- a/drivers/mmc/core/core.c
>> +++ b/drivers/mmc/core/core.c
>> @@ -1482,9 +1482,10 @@ void mmc_rescan(struct work_struct *work)
>>  			mmc_release_host(host);
>>  			goto out;
>>  		}
>> +#ifdef CONFIG_MMC_DEBUG
>>  		pr_info("%s: %s: trying to init card at %u Hz\n",
>>  			mmc_hostname(host), __func__, host->f_init);
>> -
>> +#endif
I first wrote this line while testing the patch for "configurable ID freq".
I left it in thinking it would only trigger while booting or mounting a
new card, not being aware of the MMC_CAP_NEEDS_POLL option.
Sorry for that,

Acked-by: Hein Tibosch <hein_tibosch@xxxxxxxx>

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


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux