diff mbox series

Input: cyttsp - do not force interrupt trigger

Message ID YK7iO96g+7yIC0l1@google.com
State Accepted
Commit c9d2939dda3ba121834dba053ff8ba5026f8ce37
Headers show
Series Input: cyttsp - do not force interrupt trigger | expand

Commit Message

Dmitry Torokhov May 27, 2021, 12:05 a.m. UTC
Instead of forcing interrupt trigger to be "falling edge" let's rely on the
platform to set it up according to how it is set up on a given board based
on data in device tree or ACPI tables.

Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
---

Linus, Ferruh's email has been bouncing for ages, it looks like you have
the hardware and you are already looking over another Cypress touch
controller, mind if I put you down as a maintainer for this one as well?


 drivers/input/touchscreen/cyttsp_core.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Linus Walleij May 27, 2021, 12:30 a.m. UTC | #1
On Thu, May 27, 2021 at 2:05 AM Dmitry Torokhov
<dmitry.torokhov@gmail.com> wrote:

> Instead of forcing interrupt trigger to be "falling edge" let's rely on the

> platform to set it up according to how it is set up on a given board based

> on data in device tree or ACPI tables.

>

> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>


Reviewed-by: Linus Walleij <linus.walleij@linaro.org>


> Linus, Ferruh's email has been bouncing for ages, it looks like you have

> the hardware and you are already looking over another Cypress touch

> controller, mind if I put you down as a maintainer for this one as well?


That's fine, just list me if you want, one more or less.

Yours,
Linus Walleij
diff mbox series

Patch

diff --git a/drivers/input/touchscreen/cyttsp_core.c b/drivers/input/touchscreen/cyttsp_core.c
index d9debcdeeec7..a6352d79e23d 100644
--- a/drivers/input/touchscreen/cyttsp_core.c
+++ b/drivers/input/touchscreen/cyttsp_core.c
@@ -699,8 +699,7 @@  struct cyttsp *cyttsp_probe(const struct cyttsp_bus_ops *bus_ops,
 	}
 
 	error = devm_request_threaded_irq(dev, ts->irq, NULL, cyttsp_irq,
-					  IRQF_TRIGGER_FALLING | IRQF_ONESHOT |
-					  IRQF_NO_AUTOEN,
+					  IRQF_ONESHOT | IRQF_NO_AUTOEN,
 					  "cyttsp", ts);
 	if (error) {
 		dev_err(ts->dev, "failed to request IRQ %d, err: %d\n",