login
Login
/
Register
Search
Forums
News
Blogs
Features
Site
Home
»
Mailing list archives
»
linux-kernel
»
2008
»
April
»
14
Re: [PATCH 1/3] wm97xx-core: Only schedule interrupt handler if not already scheduled
view
thread
!MAILaRCHIVE_VOTE_RePLACE
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
[view in full thread]
From:
Dmitry Torokhov <dmitry.torokhov@...>
To: Mark Brown <broonie@...>
Cc: Andrew Morton <akpm@...>, Jiri Kosina <jkosina@...>, <linux-input@...>, <linux-kernel@...>
Subject:
Re: [PATCH 1/3] wm97xx-core: Only schedule interrupt handler if not already scheduled
Date: Monday, April 14, 2008 - 2:10 pm
Hi Mark, On Mon, Apr 14, 2008 at 06:39:33PM +0100, Mark Brown wrote:
quoted text
> static irqreturn_t wm97xx_pen_interrupt(int irq, void *dev_id) > { > struct wm97xx *wm = dev_id; > > - wm->mach_ops->irq_enable(wm, 0); > - queue_work(wm->ts_workq, &wm->pen_event_work); > + if (!work_pending(&wm->pen_event_work)) { > + wm->mach_ops->irq_enable(wm, 0); > + queue_work(wm->ts_workq, &wm->pen_event_work); > + }
Given the fact that work will not be queued if it is pending anyway why is this change needed? -- Dmitry --
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to
majordomo@vger.kernel.org
More majordomo info at
http://vger.kernel.org/majordomo-info.html
Please read the FAQ at
http://www.tux.org/lkml/
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
Messages in current thread:
WM97xx touchscreen updates
, Mark Brown
, (Mon Apr 14, 1:38 pm)
[PATCH 1/3] wm97xx-core: Only schedule interrupt handler if ...
, Mark Brown
, (Mon Apr 14, 1:39 pm)
Re: [PATCH 1/3] wm97xx-core: Only schedule interrupt handler...
, Dmitry Torokhov
, (Mon Apr 14, 2:10 pm)
Re: [PATCH 1/3] wm97xx-core: Only schedule interrupt handler...
, Mark Brown
, (Tue Apr 15, 4:48 am)
Re: [PATCH 1/3] wm97xx-core: Only schedule interrupt handler...
, Dmitry Torokhov
, (Tue Apr 15, 9:02 am)
Re: [PATCH 1/3] wm97xx-core: Only schedule interrupt handler...
, Mark Brown
, (Tue Apr 15, 10:42 am)
[PATCH 2/3] wm97xx-core: Use IRQF_SAMPLE_RANDOM
, Mark Brown
, (Mon Apr 14, 1:39 pm)
[PATCH 3/3] wm97xx-core: Support use as a wakeup source
, Mark Brown
, (Mon Apr 14, 1:39 pm)
Re: [PATCH 3/3] wm97xx-core: Support use as a wakeup source
, Dmitry Torokhov
, (Mon Apr 14, 4:34 pm)
Re: [PATCH 3/3] wm97xx-core: Support use as a wakeup source
, David Brownell
, (Mon Apr 14, 8:29 pm)
Re: [PATCH 3/3] wm97xx-core: Support use as a wakeup source
, Mark Brown
, (Tue Apr 15, 8:18 am)
Navigation
Create content
Mailing list archives
Recent posts
Popular discussions
linux-kernel
:
Andy Whitcroft
Re: 2.6.23-rc6-mm1
Greg KH
[GIT PATCH] driver core patches against 2.6.24
James Bottomley
Re: Integration of SCST in the mainstream Linux kernel
Alan
Re: [RFC] Heads up on sys_fallocate()
git
:
linux-netdev
:
Natalie Protasevich
[BUG] New Kernel Bugs
Gerrit Renker
[PATCH 0/37] dccp: Feature negotiation - last call for comments
Jarek Poplawski
[PATCH] pkt_sched: Destroy gen estimators under rtnl_lock().
Winkler, Tomas
RE: iwlwifi: fix build bug in "iwlwifi: fix LED stall"
openbsd-misc
:
Colocation donated by:
Who's online
There are currently
0 users
and
439 guests
online.
Syndicate