Hi Philip, On Sun, May 22 2011, Philip Rakity wrote: > diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c > index baab027..6db91d6 100644 > --- a/drivers/mmc/core/mmc.c > +++ b/drivers/mmc/core/mmc.c > @@ -174,14 +174,17 @@ static int mmc_decode_csd(struct mmc_card *card) > } > > /* > - * Read and decode extended CSD. > + * Read extended CSD. > */ > -static int mmc_read_ext_csd(struct mmc_card *card) > +static int mmc_read_ext_csd(struct mmc_card *card, u8 *ext_csd) This one doesn't work -- you're missing the half of the patch that hooks up the rest of mmc.c to the new code, which was in v1. As-is, we get: drivers/mmc/core/mmc.c: In function âmmc_init_cardâ: drivers/mmc/core/mmc.c:621:3: error: too few arguments to function âmmc_read_ext_csdâ drivers/mmc/core/mmc.c:240:12: note: declared here drivers/mmc/core/mmc.c: At top level: drivers/mmc/core/mmc.c:404:12: warning: âmmc_compare_ext_csdsâ defined but not used [-Wunused-function] - Chris. -- Chris Ball <cjb@xxxxxxxxxx> <http://printf.net/> One Laptop Per Child -- 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