[PATCH 4/4 V2] mmc: esdhc: Add broken timeout quirk for p4/p5 board

Zhang Haijun-B42677 B42677 at freescale.com
Fri Jul 19 12:19:59 EST 2013



Thanks.

Regards
Haijun.


> -----Original Message-----
> From: Wood Scott-B07421
> Sent: Thursday, July 18, 2013 1:14 AM
> To: Zhang Haijun-B42677
> Cc: linux-mmc at vger.kernel.org; linuxppc-dev at lists.ozlabs.org;
> cbouatmailru at gmail.com; cjb at laptop.org; Fleming Andy-AFLEMING; Zhang
> Haijun-B42677; Zhang Haijun-B42677
> Subject: Re: [PATCH 4/4 V2] mmc: esdhc: Add broken timeout quirk for
> p4/p5 board
> 
> On 07/17/2013 05:11:31 AM, Haijun Zhang wrote:
> > Sometimes command can't be completed within the time give in
> > eSDHC_SYSCTL[DTOCV]. So just give the max value 0x14 to avoid this
> > issue.
> >
> > Signed-off-by: Haijun Zhang <haijun.zhang at freescale.com>
> > ---
> > changes for v2:
> > 	- Rebuild patch of eSDHC host need long time to generate
> > 	 command interrupt
> >
> >  drivers/mmc/host/sdhci-of-esdhc.c | 6 ++++++
> >  1 file changed, 6 insertions(+)
> >
> > diff --git a/drivers/mmc/host/sdhci-of-esdhc.c
> > b/drivers/mmc/host/sdhci-of-esdhc.c
> > index 570bca8..30bfb5c 100644
> > --- a/drivers/mmc/host/sdhci-of-esdhc.c
> > +++ b/drivers/mmc/host/sdhci-of-esdhc.c
> > @@ -325,6 +325,12 @@ static void esdhc_of_platform_init(struct
> > sdhci_host *host)
> >
> >  	if (vvn > VENDOR_V_22)
> >  		host->quirks &= ~SDHCI_QUIRK_NO_BUSY_IRQ;
> > +
> > +	if ((SVR_SOC_VER(svr) == SVR_B4860) ||
> > +		(SVR_SOC_VER(svr) == SVR_P5020) ||
> > +		(SVR_SOC_VER(svr) == SVR_P5040) ||
> > +		(SVR_SOC_VER(svr) == SVR_P4080))
> > +		host->quirks |= SDHCI_QUIRK_BROKEN_TIMEOUT_VAL;
> >  }
> 
> Please don't line up the continuation lines of the if-condition with the
> if-body.
[Haijun Wrote:] I'll correct it.
> 
> Please check variant SoCs as well.  If the bug exists on p4080, then it
> exists on p4040.  Likewise with p5040/p5021, and p5020/p5010.
> 
> Is it present on all revisions of these SoCs?  How about p3041, which is
> usually pretty similar to p5020?  p2040/p2041?  Is there an erratum
> number for this problem?
> 
[Haijun Wrote:] I only checked this on these boards. No errata number yet, This
quirk only give the host max detecting time value to check card's response. No
impact on performance or other functions. 

> -Scott



More information about the Linuxppc-dev mailing list