Messages in this thread | | | Date | Thu, 06 Sep 2012 19:41:43 +0530 | From | Shubhrajyoti <> | Subject | Re: [PATCH] OMAP GPIO - don't wake from suspend unless requested. |
| |
On Thursday 06 September 2012 12:32 PM, NeilBrown wrote: > From: NeilBrown <neilb@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 <khilman@ti.com> > Cc: Tony Lindgren <tony@atomide.com> > Cc: Santosh Shilimkar <santosh.shilimkar@ti.com> > Cc: Cousson Benoit <b-cousson@ti.com> > Cc: Grant Likely <grant.likely@secretlab.ca> > Cc: Tarun Kanti DebBarma <tarun.kanti@ti.com> > Cc: Felipe Balbi <balbi@ti.com> > Cc: Govindraj.R <govindraj.raja@ti.com> > > Signed-off-by: NeilBrown <neilb@suse.de>
| |