On 10/10/2013 07:00 PM, Tony Lindgren wrote: > * Roger Quadros <rogerq@xxxxxx> [131010 06:32]: >> >> I tried testing this with the USB EHCI driver, but I'm not getting wake up interrupts >> while the system is still running and only the EHCI controller is runtime suspended. >> >> It seems we need to somehow call _reconfigure_io_chain() to update the daisy chain >> whenever the pad wakeup_enable bit is changed. > > Sounds like this is on omap3? Have you tried calling pcs_soc->rearm() in the > pcs_irq_handle() like the comments there suggest? At least for me that keeps > the wake-up interrupts continuously running on omap3 instead of just idle modes. Yes it is on OMAP3. I haven't tried with pcs_soc->rearm(). I will give it a try and let you know. > > Now on omap4, I've noticed the wake up interrupts are on all the time based on tests > with the serial driver. > >> I think pcs_irq_set_wake() is where need to control system wakeup behaviour for the irq. >> This is where we should be able to change WAKEUP_EN bit of the pad >> to enable/disable system wakeup for that pad and also call _reconfigure_io_chain(). > > Well the irq_set_wake() should only be needed for suspend and resume. For runtime PM > the wake-events should be always enabled by default as pointed out by Alan Stern > a while back. Right, but we need to update the WAKEUP_EN bit in the pad control register for that to work, no?. This is something we are not doing in the driver. > >> This would mean that we don't really need to set WAKEUP_EN for the pads in the DTS file. > > Well for runtime PM, we should also do the automatic handling if configured. > But how to do that best is still open.. I didn't get this part. I was thinking that irq_set_wake() should map directly to WAKEUP_EN bit for the pin. cheers, -roger -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html