Hello Ulf, On Mon, Feb 2, 2015 at 5:00 PM, Ulf Hansson <ulf.hansson@xxxxxxxxxx> wrote: > Host drivers have different ways to sends their "init stream" to the > card. Some need to do it as part of a request, some do it from the > ->set_ios() callback in the MMC_POWER_ON state and some don't send an > "init stream" at all. > > To be able to use the reset GPIOs from the simple MMC power sequence > provider, the card need to be powered and the "init stream" must not > have been sent. > > To cope with these requirements, invoke mmc_pwrseq_post_power_on() > prior we change the state to MMC_POWER_ON in mmc_power_up(). > > Host drivers shall perform power up operations in the MMC_POWER_UP > state. Unfortunate three hosts (au1xmmc, cb710-mmc and toshsd) don't > conform to this expectation. Instead those ignore the MMC_POWER_UP > state and delays their power up operations to the MMC_POWER_ON state. > > Those hosts needs to change their behavior to enable proper support for > the simple MMC power sequence provider. > > Signed-off-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx> The patch looks good to me and I tested that the WLAN chip attached to the SDIO interface in my Exynos5250 Snow Chromebook is reset / powered up correctly so: Reviewed-by: Javier Martinez Canillas <javier.martinez@xxxxxxxxxxxxxxx> Tested-by: Javier Martinez Canillas <javier.martinez@xxxxxxxxxxxxxxx> Best regards, Javier -- 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
