Thanks, Anton. I will post it again. > -----Original Message----- > From: Anton Vorontsov [mailto:cbouatmailru@xxxxxxxxx] > Sent: Friday, January 13, 2012 7:55 PM > To: Huang Changming-R66093 > Cc: Chris Ball; linux-mmc@xxxxxxxxxxxxxxx > Subject: Re: [PATCH 2/5 v2] ESDHC: set the timeout to the max value > > On Fri, Jan 13, 2012 at 02:36:21AM +0000, Huang Changming-R66093 wrote: > > Hi, Anton, > > Could you have any comment about this serial patch? > > > > Thanks > > Jerry Huang > > > > > -----Original Message----- > > > From: Huang Changming-R66093 > > > Sent: Friday, January 06, 2012 1:33 PM > > > To: linux-mmc@xxxxxxxxxxxxxxx > > > Cc: Huang Changming-R66093; Gao Guanhua-B22826; Xie Xiaobo-R63061; > > > Chris Ball > > > Subject: [PATCH 2/5 v2] ESDHC: set the timeout to the max value > > > > > > From: Jerry Huang <Chang-Ming.Huang@xxxxxxxxxxxxx> > > > > > > When access the card on some FSL platform board (e.g p2020, p1010, > > > mpc8536), the following error is reported with the timeout value > > > calculated: > > > > > > mmc0: Got data interrupt 0x00000020 even though no data operation > > > was in progress. > > > mmc0: Got data interrupt 0x00000020 even though no data operation > > > was in progress. > > > > > > So we skip the calculation of timeout and use the max value to fix it. > > > > > > Signed-off-by: Gao Guanhua <B22826@xxxxxxxxxxxxx> > > > Signed-off-by: Xie Xiaobo <X.Xie@xxxxxxxxxxxxx> > > > Signed-off-by: Jerry Huang <Chang-Ming.Huang@xxxxxxxxxxxxx> > > > CC: Chris Ball <cjb@xxxxxxxxxx> > > > --- > > > changes for v2: > > > - change the property to compatible for quirks > > > > > > drivers/mmc/host/sdhci-pltfm.c | 5 +++++ > > > 1 files changed, 5 insertions(+), 0 deletions(-) > > > > > > diff --git a/drivers/mmc/host/sdhci-pltfm.c > > > b/drivers/mmc/host/sdhci- pltfm.c index ba36c36..1c254b1 100644 > > > --- a/drivers/mmc/host/sdhci-pltfm.c > > > +++ b/drivers/mmc/host/sdhci-pltfm.c > > > @@ -74,6 +74,11 @@ void sdhci_get_of_property(struct platform_device > > > *pdev) > > > if (of_device_is_compatible(np, "fsl,p2020-rev1-esdhc")) > > > host->quirks |= SDHCI_QUIRK_BROKEN_DMA; > > > > > > + if (of_device_is_compatible(np, "fsl,p2020-esdhc") > > > + || of_device_is_compatible(np, "fsl,p1010-esdhc") > > > + || of_device_is_compatible(np, "fsl,mpc8536-esdhc")) > > A would add one more tab on these two lines. > > Otherwise, looks OK. > > Acked-by: Anton Vorontsov <cbouatmailru@xxxxxxxxx> > > Thanks, > > > > + host->quirks |= SDHCI_QUIRK_BROKEN_TIMEOUT_VAL; > > > + > > -- > Anton Vorontsov > Email: cbouatmailru@xxxxxxxxx ��.n��������+%������w��{.n�����{��i��)�� jg��� �����ݢj����G���� ���j:+v���w�m������w�������h�����٥
