* [PATCH omap-fixes v2] OMAP2/3: GPIO: do not attempt to wake-enable GPIO bank IRQ @ 2009-03-09 22:14 Kevin Hilman 2009-03-18 22:07 ` [APPLIED] [PATCH omap-fixes v2] OMAP2/3: GPIO: do not attempt to wake-enable Tony Lindgren 0 siblings, 1 reply; 7+ messages in thread From: Kevin Hilman @ 2009-03-09 22:14 UTC (permalink / raw) To: linux-omap The GPIO IRQ enable/disable path attempts to also enable IRQ wake support for the parent GPIO bank IRQ as well. However, since there is no 'set_wake' hook for the bank IRQs, these calls will always fail. Also, since the enable will fail on the suspend path, the disable on the resume path will trigger unbalanced enable/disable warnings. This was discovered in the suspend/resume path on OMAP3/Beagle using the gpio-keys driver which disables/re-enables GPIO IRQ wakeups in the suspend/resume path. Signed-off-by: Kevin Hilman <khilman@deeprootsystems.com> --- This patch was sent on 28 jan 2009 as: [PATCH omap-fixes] OMAP2/3: GPIO: remove recursion in IRQ wakeup path This version simply updates the description to better describe the problem and solution. arch/arm/plat-omap/gpio.c | 14 ++++---------- 1 files changed, 4 insertions(+), 10 deletions(-) diff --git a/arch/arm/plat-omap/gpio.c b/arch/arm/plat-omap/gpio.c index f856a90..798a8cd 100644 --- a/arch/arm/plat-omap/gpio.c +++ b/arch/arm/plat-omap/gpio.c @@ -837,13 +837,10 @@ static int _set_gpio_wakeup(struct gpio_bank *bank, int gpio, int enable) case METHOD_MPUIO: case METHOD_GPIO_1610: spin_lock_irqsave(&bank->lock, flags); - if (enable) { + if (enable) bank->suspend_wakeup |= (1 << gpio); - enable_irq_wake(bank->irq); - } else { - disable_irq_wake(bank->irq); + else bank->suspend_wakeup &= ~(1 << gpio); - } spin_unlock_irqrestore(&bank->lock, flags); return 0; #endif @@ -856,13 +853,10 @@ static int _set_gpio_wakeup(struct gpio_bank *bank, int gpio, int enable) return -EINVAL; } spin_lock_irqsave(&bank->lock, flags); - if (enable) { + if (enable) bank->suspend_wakeup |= (1 << gpio); - enable_irq_wake(bank->irq); - } else { - disable_irq_wake(bank->irq); + else bank->suspend_wakeup &= ~(1 << gpio); - } spin_unlock_irqrestore(&bank->lock, flags); return 0; #endif -- 1.6.1.2 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [APPLIED] [PATCH omap-fixes v2] OMAP2/3: GPIO: do not attempt to wake-enable 2009-03-09 22:14 [PATCH omap-fixes v2] OMAP2/3: GPIO: do not attempt to wake-enable GPIO bank IRQ Kevin Hilman @ 2009-03-18 22:07 ` Tony Lindgren 2009-03-19 16:43 ` Kevin Hilman 0 siblings, 1 reply; 7+ messages in thread From: Tony Lindgren @ 2009-03-18 22:07 UTC (permalink / raw) To: linux-omap This patch has been applied to the linux-omap by youw fwiendly patch wobot. Commit: 2ac496a208895c925aec1774a873b5b096b2d3f0 PatchWorks http://patchwork.kernel.org/patch/10719/ Git http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap-2.6.git;a=commit;h=2ac496a208895c925aec1774a873b5b096b2d3f0 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [APPLIED] [PATCH omap-fixes v2] OMAP2/3: GPIO: do not attempt to wake-enable 2009-03-18 22:07 ` [APPLIED] [PATCH omap-fixes v2] OMAP2/3: GPIO: do not attempt to wake-enable Tony Lindgren @ 2009-03-19 16:43 ` Kevin Hilman 2009-03-19 16:53 ` Tony Lindgren 0 siblings, 1 reply; 7+ messages in thread From: Kevin Hilman @ 2009-03-19 16:43 UTC (permalink / raw) To: Tony Lindgren; +Cc: linux-omap Tony Lindgren <tony@atomide.com> writes: > This patch has been applied to the linux-omap > by youw fwiendly patch wobot. > > Commit: 2ac496a208895c925aec1774a873b5b096b2d3f0 > > PatchWorks > http://patchwork.kernel.org/patch/10719/ > > Git > http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap-2.6.git;a=commit;h=2ac496a208895c925aec1774a873b5b096b2d3f0 Tony, $SUBJECT on this commit seems to have been a bit mangled leaving a not very clear commit log/history. Kevin ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [APPLIED] [PATCH omap-fixes v2] OMAP2/3: GPIO: do not attempt to wake-enable 2009-03-19 16:43 ` Kevin Hilman @ 2009-03-19 16:53 ` Tony Lindgren 2009-03-19 16:48 ` Felipe Balbi 2009-03-19 17:16 ` Tony Lindgren 0 siblings, 2 replies; 7+ messages in thread From: Tony Lindgren @ 2009-03-19 16:53 UTC (permalink / raw) To: Kevin Hilman; +Cc: linux-omap * Kevin Hilman <khilman@deeprootsystems.com> [090319 09:44]: > Tony Lindgren <tony@atomide.com> writes: > > > This patch has been applied to the linux-omap > > by youw fwiendly patch wobot. > > > > Commit: 2ac496a208895c925aec1774a873b5b096b2d3f0 > > > > PatchWorks > > http://patchwork.kernel.org/patch/10719/ > > > > Git > > http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap-2.6.git;a=commit;h=2ac496a208895c925aec1774a873b5b096b2d3f0 > > Tony, > > $SUBJECT on this commit seems to have been a bit mangled leaving > a not very clear commit log/history. Heh looks like bad html parsing :) I'll revert and re-apply. Tony ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [APPLIED] [PATCH omap-fixes v2] OMAP2/3: GPIO: do not attempt to wake-enable 2009-03-19 16:53 ` Tony Lindgren @ 2009-03-19 16:48 ` Felipe Balbi 2009-03-19 17:16 ` Tony Lindgren 1 sibling, 0 replies; 7+ messages in thread From: Felipe Balbi @ 2009-03-19 16:48 UTC (permalink / raw) To: ext Tony Lindgren; +Cc: Kevin Hilman, linux-omap On Thu, Mar 19, 2009 at 05:53:35PM +0100, Tony Lindgren wrote: > * Kevin Hilman <khilman@deeprootsystems.com> [090319 09:44]: > > Tony Lindgren <tony@atomide.com> writes: > > > > > This patch has been applied to the linux-omap > > > by youw fwiendly patch wobot. > > > > > > Commit: 2ac496a208895c925aec1774a873b5b096b2d3f0 > > > > > > PatchWorks > > > http://patchwork.kernel.org/patch/10719/ > > > > > > Git > > > http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap-2.6.git;a=commit;h=2ac496a208895c925aec1774a873b5b096b2d3f0 > > > > Tony, > > > > $SUBJECT on this commit seems to have been a bit mangled leaving > > a not very clear commit log/history. > > Heh looks like bad html parsing :) I'll revert and re-apply. why do you even try to parse the html ?? patchworks hold it in mbox format: http://patchwork.kernel.org/patch/10719/mbox/ -- balbi ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [APPLIED] [PATCH omap-fixes v2] OMAP2/3: GPIO: do not attempt to wake-enable 2009-03-19 16:53 ` Tony Lindgren 2009-03-19 16:48 ` Felipe Balbi @ 2009-03-19 17:16 ` Tony Lindgren 2009-03-19 17:27 ` Felipe Balbi 1 sibling, 1 reply; 7+ messages in thread From: Tony Lindgren @ 2009-03-19 17:16 UTC (permalink / raw) To: Kevin Hilman; +Cc: linux-omap * Tony Lindgren <tony@atomide.com> [090319 09:54]: > * Kevin Hilman <khilman@deeprootsystems.com> [090319 09:44]: > > Tony Lindgren <tony@atomide.com> writes: > > > > > This patch has been applied to the linux-omap > > > by youw fwiendly patch wobot. > > > > > > Commit: 2ac496a208895c925aec1774a873b5b096b2d3f0 > > > > > > PatchWorks > > > http://patchwork.kernel.org/patch/10719/ > > > > > > Git > > > http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap-2.6.git;a=commit;h=2ac496a208895c925aec1774a873b5b096b2d3f0 > > > > Tony, > > > > $SUBJECT on this commit seems to have been a bit mangled leaving > > a not very clear commit log/history. > > Heh looks like bad html parsing :) I'll revert and re-apply. Seems like a bug in pwclient.. $ pwclient get 10719 Shows the subject in the patch as: Subject: [omap-fixes, v2] OMAP2/3: GPIO: do not attempt to wake-enable GPIO bank IRQ Tony ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [APPLIED] [PATCH omap-fixes v2] OMAP2/3: GPIO: do not attempt to wake-enable 2009-03-19 17:16 ` Tony Lindgren @ 2009-03-19 17:27 ` Felipe Balbi 0 siblings, 0 replies; 7+ messages in thread From: Felipe Balbi @ 2009-03-19 17:27 UTC (permalink / raw) To: Tony Lindgren; +Cc: Kevin Hilman, linux-omap On Thu, Mar 19, 2009 at 10:16:33AM -0700, Tony Lindgren wrote: > Seems like a bug in pwclient.. > > $ pwclient get 10719 > > Shows the subject in the patch as: > > Subject: [omap-fixes, > v2] OMAP2/3: GPIO: do not attempt to wake-enable GPIO bank IRQ $ curl http://patchwork.kernel.org/patch/10719/mbox | git am -s would do the trick :-p -- balbi ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2009-03-19 17:27 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2009-03-09 22:14 [PATCH omap-fixes v2] OMAP2/3: GPIO: do not attempt to wake-enable GPIO bank IRQ Kevin Hilman 2009-03-18 22:07 ` [APPLIED] [PATCH omap-fixes v2] OMAP2/3: GPIO: do not attempt to wake-enable Tony Lindgren 2009-03-19 16:43 ` Kevin Hilman 2009-03-19 16:53 ` Tony Lindgren 2009-03-19 16:48 ` Felipe Balbi 2009-03-19 17:16 ` Tony Lindgren 2009-03-19 17:27 ` Felipe Balbi
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).