On Wed, 7 Jul 2010, Roger Quadros wrote: > On 07/06/2010 10:51 PM, Hunter Adrian (Nokia-MS/Helsinki) wrote: > > For eMMC in omap_hsmmc, this is all done via claim_host / release_host > > which call ->enable() / ->disable() methods. omap_hsmmc makes use of > > mmc_power_restore_host() which calls host->bus_ops->power_restore() > > which is not implemented for SDIO, but for MMC and SD it reinitializes > > the card. This is IMHO a really bad design. The power control decision has to come from the top, not from the bottom. And certainly not with a U-turn dependency the omap_hsmmc is using. I regret to say this, but the omap_hsmmc driver is becoming a total mess. The host controller driver has to be a dumb interface serving requests from the hardware used by the upper layer stack, not the place where decisions such as power handling should be made. Think of it like an ethernet driver. No ethernet driver in Linux is telling the IP stack when to shut down. > Shouldn't the power control intelligence (i.e. when to turn power ON/OFF) lie > with the bus drivers? Absolutely! And in the SDIO case that should lie with each function drivers. Please let's stop this omap_hsmmc madness. Nicolas -- 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
