Thread (96 messages) 96 messages, 12 authors, 2009-07-22

Re: [PATCH 1/2] input: misc: add twl4030-pwrbutton driver

From: Felipe Balbi <hidden>
Date: 2009-03-04 10:29:35
Also in: lkml

On Wed, Mar 04, 2009 at 10:00:27AM +0100, ext Dmitry Torokhov wrote:
On Sun, Mar 01, 2009 at 04:40:19PM +0200, Felipe Balbi wrote:
quoted
On Sat, Feb 28, 2009 at 04:58:01PM -0800, Dmitry Torokhov wrote:
quoted
On Sun, Mar 01, 2009 at 02:30:18AM +0200, Felipe Balbi wrote:
quoted
On Sat, Feb 28, 2009 at 02:23:03PM -0800, Dmitry Torokhov wrote:
quoted
Hi Felipe,

On Fri, Feb 27, 2009 at 09:28:02PM +0200, Felipe Balbi wrote:
quoted
From: Felipe Balbi <redacted>

This is part of the twl4030 multifunction device driver.

With this driver we add support for reporting KEY_POWER
events via the input layer.
...
thanks for reviewing, how about the version below:
Looks good, couple more items...
Fixed the extra comments and also got rid of the global input_dev
structure. It could be fetched by passing it to request_irq() and
platform_set/get_drvdata():
Cool, I will keep it in my tree pending the threaded IRQ issue
resolution.
I guess the threaded IRQ issue will still take a while to be solved and
this driver is working as is, so there's no big problem in pushing it
and fixing the threaded irq on later patches. But I'd like to hear from
Andrew, Dave and the others about that.

-- 
balbi
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help