Hi Madhav, The patch is extremely whitespace-damaged, and your code is also formatted incorrectly. Some general points: * have you measured whether there's a performance impact from bringing the clock up and down for every request? * I'd expect this to be implemented as an MMC capability, so that it can be avoided on specific controllers. Thanks, - Chris. -- Chris Ball <cjb@xxxxxxxxxx> 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