> > > OK, so it looks like your box refuses to set up one of the GPIOs as a > > > wakeup source... Hmm, either your box is wrong ;) or matrix_keypad > > > driver needs to maintain a separate list of wakeup GPIOs. > > > > > > > This is due to the nature of PXA processor, where not every GPIO can > > be configured as a wakeup source. Mmm.... we can either return a > > pseudo value indicating setting wakeup on that GPIO is OK (which > > doesn't sound like a good idea), or we can just ignore the failure of > > enable_irq_wake() in matrix_keypad? > > We ignore the failure right now in the mainline but that causes stack > traces on resume as we trying to disable not enabled wakeup GPIOs. That > was original Pavel's complaint. Yep... I'd say that BUG() simply should not trigger if wakeup can not be enabled/disabled for particular source...? Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
