Hi Florian 2014-08-24 10:26 GMT+02:00 Florian Vaussard <florian.vaussard@xxxxxxx>: > Hi Andreas, > > On 05/29/2014 10:28 AM, Andreas Fenkart wrote: >> There have been various patches floating around for enabling >> the SDIO IRQ for hsmmc, but none of them ever got merged. >> > > [...] > >> For now, only support SDIO interrupt if we are booted with >> a separate wake-irq configued via device tree. This is >> because omaps need the wake-irq for idle states, and some >> omaps need special quirks. And we don't want to add new >> legacy mux platform init code callbacks any longer as we >> are moving to DT based booting anyways. >> >> To use it, you need to specify the wake-irq using the >> interrupts-extended property. >> > > First, thanks a lot for your tenacity on this patchset, this was a long > needed feature. I enabled the SDIO interrupt, and got the throughput of > my 88W8686-based chip multiplied by 15. Nice! I just have an issue with > the wake-up path, and maybe you could help me. > > According to the DM3730 TRM, the MMC2 has the SWAKEUP path. So first I > tried to give the same wake-irq as the MMC's one, but > omap_hsmmc_configure_wake_irq() fails to request it, as they are not > IRQF_SHARED. Why can't it be shared? > So I used the DAT1 for the wake-irq (see patch below), and things are > working. But I get ~2000 wake-irq per seconds, even without any activity > on the WiFi. As a result, the driver is ping-ponging between > omap_hsmmc_runtime_suspend() and omap_hsmmc_runtime_resume(), causing > kworker to eat most of my CPU. > > Am I missing something obvious? > > Thanks! > Florian > > -- 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
