Input: adxl34x - do not hardcode interrupt trigger type
authorMarek Vasut <marex@denx.de>
Thu, 11 May 2023 00:27:55 +0000 (17:27 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 19 Jul 2023 14:21:17 +0000 (16:21 +0200)
[ Upstream commit e96220bce5176ed2309f77f061dcc0430b82b25e ]

Instead of hardcoding IRQ trigger type to IRQF_TRIGGER_HIGH, let's
respect the settings specified in the firmware description.

Fixes: e27c729219ad ("Input: add driver for ADXL345/346 Digital Accelerometers")
Signed-off-by: Marek Vasut <marex@denx.de>
Acked-by: Michael Hennerich <michael.hennerich@analog.com>
Link: https://lore.kernel.org/r/20230509203555.549158-1-marex@denx.de
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/input/misc/adxl34x.c

index a4af314..69e359f 100644 (file)
@@ -811,8 +811,7 @@ struct adxl34x *adxl34x_probe(struct device *dev, int irq,
        AC_WRITE(ac, POWER_CTL, 0);
 
        err = request_threaded_irq(ac->irq, NULL, adxl34x_irq,
-                                  IRQF_TRIGGER_HIGH | IRQF_ONESHOT,
-                                  dev_name(dev), ac);
+                                  IRQF_ONESHOT, dev_name(dev), ac);
        if (err) {
                dev_err(dev, "irq %d busy?\n", ac->irq);
                goto err_free_mem;