On Thursday 06 September 2012 12:32 PM, NeilBrown wrote: > From: NeilBrown <ne...@suse.de> > Subject: [PATCH] OMAP GPIO - don't wake from suspend unless requested. > > Current kernel will wake from suspend on an event on any active > GPIO even if enable_irq_wake() wasn't called. > > There are two reasons that the hardware wake-enable bit should be set: > > 1/ while non-suspended the CPU might go into a deep sleep (off_mode) > in which the wake-enable bit is needed for an interrupt to be > recognised. > 2/ while suspended the GPIO interrupt should wake from suspend if and > only if irq_wake as been enabled. > > The code currently doesn't keep these two reasons separate so they get > confused and sometimes the wakeup flags is set incorrectly. > > This patch reverts: > commit 9c4ed9e6c01e7a8bd9079da8267e1f03cb4761fc > gpio/omap: remove suspend/resume callbacks > and > commit 0aa2727399c0b78225021413022c164cb99fbc5e > gpio/omap: remove suspend_wakeup field from struct gpio_bank > > and makes some minor changes so that we have separate flags for "GPIO > should wake from deep idle" and "GPIO should wake from suspend". > > With this patch, the GPIO from my touch screen doesn't wake my device > any more, which is what I want. Just gave it a try fixes a similar issue on my omap4 board as well. > > Cc: Kevin Hilman <khil...@ti.com> > Cc: Tony Lindgren <t...@atomide.com> > Cc: Santosh Shilimkar <santosh.shilim...@ti.com> > Cc: Cousson Benoit <b-cous...@ti.com> > Cc: Grant Likely <grant.lik...@secretlab.ca> > Cc: Tarun Kanti DebBarma <tarun.ka...@ti.com> > Cc: Felipe Balbi <ba...@ti.com> > Cc: Govindraj.R <govindraj.r...@ti.com> > > Signed-off-by: NeilBrown <ne...@suse.de>
-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
