Hi, I am not an SDIO expert. But what I understand from this is that we are missing the wakeup capability on an SDIO interrupt. Regards, Madhu > -----Original Message----- > From: John Rigby [mailto:jcrigby@xxxxxxxxx] > Sent: Wednesday, January 27, 2010 5:56 PM > To: Madhusudhan > Cc: Mike Rapoport; Dirk Behme; Phaneendra Kumar Alapati; linux- > omap@xxxxxxxxxxxxxxx; linux-mmc@xxxxxxxxxxxxxxx; Adrian Hunter; Paasikivi > Teemu.3 (EXT-Ixonos/Tampere) > Subject: Re: sdio irq hack for omap_hsmmc.c (was: Re: [PATCH] > OMAP35xx:SDIO IRQ Support for OMAP35xx) > > Just to be clear, my post was in response to the reported problems > with enabling sdio irqs with this patch: > http://patchwork.kernel.org/patch/69122/ > > The patch originator reported good results but others reported > complete failure. My hack fixed the problem for me. I believe this > much more modest patch will also fix the problem but it is untested > because I don't have a board running right now. Beware, I'm sending > this from gmail so it may get line wrapped but it is a trivial patch > so I think it is understandable. > > John > > diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c > index fa94580..7af970e 100644 > --- a/drivers/mmc/host/omap_hsmmc.c > +++ b/drivers/mmc/host/omap_hsmmc.c > @@ -1727,10 +1727,16 @@ static int __init omap_hsmmc_probe > > omap_hsmmc_context_save(host); > > +#if 0 > mmc->caps |= MMC_CAP_DISABLE; > mmc_set_disable_delay(mmc, OMAP_MMC_DISABLED_TIMEOUT); > /* we start off in DISABLED state */ > host->dpm_state = DISABLED; > +#else > + /* we start off in ENABLED state */ > + host->dpm_state = ENABLED; > + clk_enable(host->fclk); > +#endif > > if (mmc_host_enable(host->mmc) != 0) { > clk_put(host->iclk); > > > On Wed, Jan 27, 2010 at 4:14 PM, Madhusudhan <madhu.cr@xxxxxx> wrote: > > It is hard to comment on what is happening here without a patch. Is it a > > performance issue or that SDIO functionality failure with PM enabled? > > > > The MMC controller supports wakeup capability on the dat1 line using a > GPIO > > for a SDIO interrupt. I guess that should help. > > > > Regards, > > Madhu > > -- 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
