> -----Original Message----- > From: kyungmin78@xxxxxxxxx [mailto:kyungmin78@xxxxxxxxx] On Behalf Of > Kyungmin Park > Sent: Wednesday, July 14, 2010 6:09 PM > To: Ghorai, Sukumar > Cc: Sripathy, Vishwanath; linux-mmc@xxxxxxxxxxxxxxx; linux- > omap@xxxxxxxxxxxxxxx > Subject: Re: [PATCH] mmc: csd version check updated to support MMC v4.41 > > Hi, > > you can find it's already merged at mmc tree. > > Try to find this one. [PATCH v2] Recognize CSD structure version 3 [Ghorai] Thanks I got it. > > Thank you, > Kyungmin Park > > On Wed, Jul 14, 2010 at 9:19 PM, Ghorai, Sukumar <s-ghorai@xxxxxx> wrote: > > > > > >> -----Original Message----- > >> From: Sripathy, Vishwanath > >> Sent: Wednesday, July 14, 2010 5:23 PM > >> To: Ghorai, Sukumar; linux-mmc@xxxxxxxxxxxxxxx; linux- > omap@xxxxxxxxxxxxxxx > >> Subject: RE: [PATCH] mmc: csd version check updated to support MMC > v4.41 > >> > >> > >> > >> > -----Original Message----- > >> > From: linux-omap-owner@xxxxxxxxxxxxxxx [mailto:linux-omap- > >> > owner@xxxxxxxxxxxxxxx] On Behalf Of Ghorai, Sukumar > >> > Sent: Wednesday, July 14, 2010 1:51 PM > >> > To: linux-mmc@xxxxxxxxxxxxxxx; linux-omap@xxxxxxxxxxxxxxx > >> > Cc: Ghorai, Sukumar > >> > Subject: [PATCH] mmc: csd version check updated to support MMC v4.41 > >> > > >> > CSD_STRUCTURE [127:126] describes the version of the CSD structure. > >> > According to the MMC specificaiton (v4.4.1), 3 is also a valid > number. > >> > > >> > Signed-off-by: Sukumar Ghorai <s-ghorai@xxxxxx> > >> > --- > >> > Tested on omap4430 ES2.0 > >> > > >> > drivers/mmc/core/mmc.c | 2 +- > >> > 1 files changed, 1 insertions(+), 1 deletions(-) > >> > > >> > diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c > >> > index 89f7a25..525af33 100644 > >> > --- a/drivers/mmc/core/mmc.c > >> > +++ b/drivers/mmc/core/mmc.c > >> > @@ -122,7 +122,7 @@ static int mmc_decode_csd(struct mmc_card *card) > >> > * v1.2 has extra information in bits 15, 11 and 10. > >> > */ > >> > csd_struct = UNSTUFF_BITS(resp, 126, 2); > >> > - if (csd_struct != 1 && csd_struct != 2) { > >> > + if (csd_struct > 3) { > >> > >> Can't csd_struct be 0? If so then your new check will break right? > > [Ghorai] 0 to 3 are the valid numbers. > > > >> > >> > printk(KERN_ERR "%s: unrecognised CSD structure > version %d\n", > >> > mmc_hostname(card->host), csd_struct); > >> > return -EINVAL; > >> > -- > >> > To unsubscribe from this list: send the line "unsubscribe linux-omap" > in > >> > the body of a message to majordomo@xxxxxxxxxxxxxxx > >> > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- > > 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 > > -- 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
