[PATCH] input: tca6416-keypad: add IRQF_ONESHOT IRQ flag

Subsystems: input (keyboard, mouse, joystick, touchscreen) drivers, the rest

STALE5217d

2 messages, 2 authors, 2012-07-04 · open the first message on its own page

[PATCH] input: tca6416-keypad: add IRQF_ONESHOT IRQ flag

From: Guennadi Liakhovetski <hidden>
Date: 2012-07-04 10:19:15

request_threaded_irq() without a hard IRQ handler can only be used, if the
driver also requests to keep the interrupt masked by using the IRQF_ONESHOT
IRQ flag.

Signed-off-by: Guennadi Liakhovetski <redacted>
---

This has to go to 3.5 to fix otherwise broken platforms.

 drivers/input/keyboard/tca6416-keypad.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/input/keyboard/tca6416-keypad.c b/drivers/input/keyboard/tca6416-keypad.c
index 3afea3f..4559859 100644
--- a/drivers/input/keyboard/tca6416-keypad.c
+++ b/drivers/input/keyboard/tca6416-keypad.c
@@ -278,7 +278,7 @@ static int __devinit tca6416_keypad_probe(struct i2c_client *client,
 
 		error = request_threaded_irq(chip->irqnum, NULL,
 					     tca6416_keys_isr,
-					     IRQF_TRIGGER_FALLING,
+					     IRQF_TRIGGER_FALLING | IRQF_ONESHOT,
 					     "tca6416-keypad", chip);
 		if (error) {
 			dev_dbg(&client->dev,
-- 
1.7.2.5

Re: [PATCH] input: tca6416-keypad: add IRQF_ONESHOT IRQ flag

From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Date: 2012-07-04 20:19:08

Hi Guennadi,

On Wed, Jul 04, 2012 at 12:19:13PM +0200, Guennadi Liakhovetski wrote:
request_threaded_irq() without a hard IRQ handler can only be used, if the
driver also requests to keep the interrupt masked by using the IRQF_ONESHOT
IRQ flag.
Thank you for the patch but I already applied one from Lars-Peter
Clausen.

Thanks.

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