On 05/02/12 03:09, Chris Ball wrote: > This patch doesn't look correct to me -- yes, the hardware might have > lost power during suspend, but that's what the common-case of: > > sdhci_init(host, soft=0); > > is for. soft=1 is supposed to indicate that the power stayed up via > MMC_PM_KEEP_POWER, but you're changing that by setting ->pwr = 0 in the > soft=1 path. If I'm using soft=1 because my controller has a wifi SDIO > card on it, I don't want you to reprogram the power to it on resume. > Does that make sense? Yes, I will send a different patch (to handle the "card is on, but the host controller is off" case) -- 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