Thanks Jerry Huang
> -----Original Message----- > From: Chris Ball [mailto:[email protected]] > Sent: Tuesday, February 14, 2012 4:44 AM > To: Huang Changming-R66093 > Cc: [email protected]; Jain Priyanka-B32167 > Subject: Re: [PATCH v7] ESDHC: Workaround for data crc error on p1010rdb > > Hi, > > On Mon, Feb 13 2012, Huang Changming-R66093 wrote: > >> Actually, I think I'm still missing something. Why doesn't this work? > >> > >> +static void esdhc_of_set_clock(struct sdhci_host *host, unsigned int > >> +clock) { > >> + /* Workaround to reduce the clock frequency for p1010 esdhc */ > >> + if (of_find_compatible_node(NULL, NULL, "fsl,p1010-esdhc")) { > >> + if (clock > 20000000) > >> + clock -= 5000000; > >> + if (clock > 40000000) > >> + clock -= 5000000; > >> + } > >> + > >> + /* Set the clock */ > >> + esdhc_set_clock(host, clock); } > >> > > These codes can work, too. > > I forget the clock will be reduced 5M/10M only when the clock is > > greater than 20MHz/45MHz. > > So I should not test the clock in this function. > > Great, thanks -- please test and submit a patch with the code above, and > I'll merge it. Thanks, Chris. I have sent it again. Jerry Huang -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
