Hi, On Fri, Nov 11 2011, Ulf Hansson wrote: > Girish K S wrote: >> This patch skips the setting of the power notify state variable >> for non eMMC 4.5 devices. Also fixes the problem of omap_hsmmc >> noisy/broken for suspend resume reported by kevin Hilman. >> >> cc: Chris Ball <cjb@xxxxxxxxxx> >> Signed-off-by: Girish K S <girish.shivananjappa@xxxxxxxxxx> >> --- >> drivers/mmc/core/mmc.c | 10 +++++++--- >> 1 files changed, 7 insertions(+), 3 deletions(-) >> >> diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c >> index 3627044..4db73a9 100644 >> --- a/drivers/mmc/core/mmc.c >> +++ b/drivers/mmc/core/mmc.c >> @@ -882,10 +882,14 @@ static int mmc_init_card(struct mmc_host *host, u32 ocr, >> card->ext_csd.generic_cmd6_time); >> if (err && err != -EBADMSG) >> goto free_card; >> - } >> - if (!err) >> - card->poweroff_notify_state = MMC_POWERED_ON; >> + /* >> + * The err can be -EBADMSG or 0, >> + * so check for success and update the flag >> + */ >> + if (!err) >> + card->poweroff_notify_state = MMC_POWERED_ON; >> + } >> /* >> * Activate high speed (if supported) > > > I encountered similar problems as for omap_hsmmc for the mmci driver > (although with some ST-Ericsson internal pm_runtime patches). This > patch fixed the problem. Great! > > Acked-by: Ulf Hansson <ulf.hansson@xxxxxxxxxxxxxx> Thanks, pushed to mmc-next for 3.2. - 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
