diff mbox

input: tca6416-keypad: add IRQF_ONESHOT IRQ flag

Message ID Pine.LNX.4.64.1207041217420.21042@axis700.grange (mailing list archive)
State New, archived
Headers show

Commit Message

Guennadi Liakhovetski July 4, 2012, 10:19 a.m. UTC
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 <g.liakhovetski@gmx.de>
---

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(-)

Comments

Dmitry Torokhov July 4, 2012, 8:19 p.m. UTC | #1
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.
diff mbox

Patch

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,