It is much cleaner solution then before ! I like it. Reviewed-by: Philip Rakity <prakity@xxxxxxxxxxx> On Jul 18, 2012, at 7:41 PM, Chris Ball wrote: > Hi, > > On Wed, Jul 18 2012, Aaron Lu wrote: >> I've prepared the following code, please check if this is better than >> the current one: > > I think the diffstat speaks for itself. :-) > > drivers/mmc/core/sd.c | 77 ++++++++++++++++++++++++++-------------------- > drivers/mmc/host/sdhci.c | 51 ++---------------------------- > include/linux/mmc/host.h | 22 +++---------- > 3 files changed, 51 insertions(+), 99 deletions(-) > > Girish/Philip, does this look good to you too? If so, I'll squash it > into the original patch. > > Thanks! > > - 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