On Mon, 2011-02-07 at 16:56 +0200, ext Ohad Ben-Cohen wrote: > On Mon, Feb 7, 2011 at 2:58 PM, Juuso Oikarinen > <juuso.oikarinen@xxxxxxxxx> wrote: > > Actually this I don't see any performance impact for better or worse of > > this patch on brief testing. I was instructed by some of our other > > kernel-dudes that this is actually needed for the cpuidle path too, or > > there might be some implications. > > It might mean your platforms are configured to hit retention (assuming > omap) or even off mode in the cpuidle path as well, and in that case > it can explain the enable_irq_wake.. > > > I'm currently trying to find out if this is really needed or not. > > Sure, thanks. They are saying this is needed for OMAP's retention mode too, so AFAIK this is needed, not just for suspend. -Juuso -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html