On 6 May 2015 at 13:09, Adrian Hunter <adrian.hunter@xxxxxxxxx> wrote:
> On 04/05/15 16:55, Ulf Hansson wrote:
>> On 20 April 2015 at 14:09, Adrian Hunter <adrian.hunter@xxxxxxxxx> wrote:
>>> CRC or End-Bit errors could possibly be alleviated by
>>> re-tuning so flag re-tuning needed in those cases.
>>> Note this has no effect if re-tuning has not been
>>> enabled.
>>>
>>> Signed-off-by: Adrian Hunter <adrian.hunter@xxxxxxxxx>
>>> ---
>>> drivers/mmc/host/sdhci.c | 14 +++++++++-----
>>> 1 file changed, 9 insertions(+), 5 deletions(-)
>>>
>>> diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c
>>> index b345844..d11fae7 100644
>>> --- a/drivers/mmc/host/sdhci.c
>>> +++ b/drivers/mmc/host/sdhci.c
>>> @@ -2307,8 +2307,10 @@ static void sdhci_cmd_irq(struct sdhci_host *host, u32 intmask, u32 *mask)
>>> if (intmask & SDHCI_INT_TIMEOUT)
>>> host->cmd->error = -ETIMEDOUT;
>>> else if (intmask & (SDHCI_INT_CRC | SDHCI_INT_END_BIT |
>>> - SDHCI_INT_INDEX))
>>> + SDHCI_INT_INDEX)) {
>>> host->cmd->error = -EILSEQ;
>>> + mmc_retune_needed(host->mmc);
>>
>> I would rather see this to be handled by the mmc core, to have all
>> hosts benefit from it.
>
> Now that the core is enabling re-tuning, that will cause re-tuning to happen
> on platforms where it has never been used before (or more specifically where
> tuning has been used only during initialization)
>
> I have no idea if that is the right thing for all host controllers and
> platforms.
>
> Are you sure you want to do that?
This is required by the specs, so at least I think we should give it a try.
If we see issues with it, we could have it as an opt-in/out MMC_CAP.
Kind regards
Uffe
--
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
