Hi, you can find it's already merged at mmc tree. Try to find this one. [PATCH v2] Recognize CSD structure version 3 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
