Thread (6 messages) flat view 6 messages, 2 authors, 2016-11-12

Re: [PATCH] drivers: tca8418: Change the interrupt type

From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Date: 2016-11-09 00:04:32
Also in: lkml

On Mon, Nov 07, 2016 at 03:40:24PM +0100, Maxime Ripard wrote:
The TCA8418 interrupt has a level trigger, not a edge one.

Signed-off-by: Maxime Ripard <redacted>
Hmm, maybe we could rely on OF data for trigger type?
quoted hunk ↗ jump to hunk
---
 drivers/input/keyboard/tca8418_keypad.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/input/keyboard/tca8418_keypad.c b/drivers/input/keyboard/tca8418_keypad.c
index 9002298698fc..b88b3696a2e1 100644
--- a/drivers/input/keyboard/tca8418_keypad.c
+++ b/drivers/input/keyboard/tca8418_keypad.c
@@ -360,7 +360,7 @@ static int tca8418_keypad_probe(struct i2c_client *client,
 		irq = gpio_to_irq(irq);
 
 	error = devm_request_threaded_irq(dev, irq, NULL, tca8418_irq_handler,
-					  IRQF_TRIGGER_FALLING |
+					  IRQF_TRIGGER_LOW |
 						IRQF_SHARED |
 						IRQF_ONESHOT,
 					  client->name, keypad_data);
-- 
2.10.1
-- 
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