lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 19 Jun 2014 11:40:06 +0100 From: One Thousand Gnomes <gnomes@...rguk.ukuu.org.uk> To: "Li, Aubrey" <aubrey.li@...ux.intel.com> Cc: "dmitry.torokhov@...il.com" <dmitry.torokhov@...il.com>, "Rafael J. Wysocki" <rjw@...ysocki.net>, "linux-input@...r.kernel.org" <linux-input@...r.kernel.org>, LKML <linux-kernel@...r.kernel.org> Subject: Re: [PATCH] GPIO button wth wakeup attribute is supposed to wake the system up On Thu, 19 Jun 2014 08:51:25 +0800 "Li, Aubrey" <aubrey.li@...ux.intel.com> wrote: > When the wakeup attribute is set, the GPIO button is capable of > waking up the system from sleep states, including the "freeze" > sleep state. For that to work, its driver needs to pass the > IRQF_NO_SUSPEND flag to devm_request_any_context_irq(), or the > interrupt will be disabled by suspend_device_irqs() and the > system won't be woken up by it from the "freeze" sleep state. > > The suspend_device_irqs() routine is a workaround for drivers > that mishandle interrupts triggered when the devices handled > by them are suspended, so it is safe to use IRQF_NO_SUSPEND in > all drivers that don't have that problem. > > The affected/tested machines include Dell Venue 11 Pro and Asus T100TA. > > Signed-off-by: Aubrey Li <aubrey.li@...ux.intel.com> > Reviewed-by: Rafael J. Wysocki <rafael.j.wysocki@...el.com> Reviewed-by: Alan Cox <alan@...ux.intel.com> -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists