Re: [PATCH v1] mmc: sdhci-of-esdhc: fix the mmc error after sleep on ls1046ardb

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

 



Hi Yinbo,

On 30 November 2017 at 11:44, Adrian Hunter <adrian.hunter@xxxxxxxxx> wrote:
> On 28/11/17 05:57, yinbo.zhu@xxxxxxx wrote:
>> From: "yinbo.zhu" <yinbo.zhu@xxxxxxx>
>>
>> When system wakes up from sleep on ls1046ardb, the SD operation fails
>> with mmc error messages since ESDHC_TB_EN bit couldn’t be cleaned by
>> eSDHC_SYSCTL[RSTA]. It's proper to clean this bit in esdhc_reset()
>> rather than in probe.
>>
>> Signed-off-by: yinbo.zhu <yinbo.zhu@xxxxxxx>
>
> Acked-by: Adrian Hunter <adrian.hunter@xxxxxxxxx>

This one was rejected by patchwork. Could you please sort out the
problem and re-submit it.

Kind regards
Uffe

>
>> ---
>>  drivers/mmc/host/sdhci-of-esdhc.c | 12 ++++++++----
>>  1 file changed, 8 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/mmc/host/sdhci-of-esdhc.c b/drivers/mmc/host/sdhci-of-esdhc.c
>> index ac2505a95403..5ee0d86a2f4d 100644
>> --- a/drivers/mmc/host/sdhci-of-esdhc.c
>> +++ b/drivers/mmc/host/sdhci-of-esdhc.c
>> @@ -578,10 +578,18 @@ static void esdhc_clock_enable(struct sdhci_host *host, bool enable)
>>
>>  static void esdhc_reset(struct sdhci_host *host, u8 mask)
>>  {
>> +     u32 val;
>> +
>>       sdhci_reset(host, mask);
>>
>>       sdhci_writel(host, host->ier, SDHCI_INT_ENABLE);
>>       sdhci_writel(host, host->ier, SDHCI_SIGNAL_ENABLE);
>> +
>> +     if (mask & SDHCI_RESET_ALL) {
>> +             val = sdhci_readl(host, ESDHC_TBCTL);
>> +             val &= ~ESDHC_TB_EN;
>> +             sdhci_writel(host, val, ESDHC_TBCTL);
>> +     }
>>  }
>>
>>  /* The SCFG, Supplemental Configuration Unit, provides SoC specific
>> @@ -771,10 +779,6 @@ static void esdhc_init(struct platform_device *pdev, struct sdhci_host *host)
>>       pltfm_host = sdhci_priv(host);
>>       esdhc = sdhci_pltfm_priv(pltfm_host);
>>
>> -     val = sdhci_readl(host, ESDHC_TBCTL);
>> -     val &= ~ESDHC_TB_EN;
>> -     sdhci_writel(host, val, ESDHC_TBCTL);
>> -
>>       host_ver = sdhci_readw(host, SDHCI_HOST_VERSION);
>>       esdhc->vendor_ver = (host_ver & SDHCI_VENDOR_VER_MASK) >>
>>                            SDHCI_VENDOR_VER_SHIFT;
>>
>
--
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