diff options
| author |   Dmitry Osipenko <digetx@gmail.com> | 2021-06-19 00:54:52 +0300 |
|---|---|---|
| committer |   Guenter Roeck <linux@roeck-us.net> | 2021-06-19 04:09:49 -0700 |
| commit | d97fb837b8cce400892e7f0ccf4755edb225ad36 (patch) | |
| tree | bdaaf891babf2e87508d760ba70e84dddef52f6c /drivers/hwmon | |
| parent | hwmon: (pmbus/dps920ab) Delete some dead code (diff) | |
| download | linux-dev-d97fb837b8cce400892e7f0ccf4755edb225ad36.tar.xz linux-dev-d97fb837b8cce400892e7f0ccf4755edb225ad36.zip | |
hwmon: (lm90) Don't override interrupt trigger type
The lm90 driver sets interrupt trigger type to level-low. This type is
not suitable for sensors like NCT1008 that don't deassert interrupt line
until temperature is back to normal, resulting in interrupt storm. The
appropriate trigger type should come from OF device description and
currently it's overridden by the driver's trigger type. Don't specify
the trigger type in the driver code, letting interrupt core to use the
device-specific trigger type.
Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers/hwmon')
| -rw-r--r-- | drivers/hwmon/lm90.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/hwmon/lm90.c b/drivers/hwmon/lm90.c index ebbfd5f352c0..2e057fad05b4 100644 --- a/drivers/hwmon/lm90.c +++ b/drivers/hwmon/lm90.c @@ -1908,8 +1908,7 @@ static int lm90_probe(struct i2c_client *client) dev_dbg(dev, "IRQ: %d\n", client->irq); err = devm_request_threaded_irq(dev, client->irq, NULL, lm90_irq_thread, - IRQF_TRIGGER_LOW | IRQF_ONESHOT, - "lm90", client); + IRQF_ONESHOT, "lm90", client); if (err < 0) { dev_err(dev, "cannot request IRQ %d\n", client->irq); return err; |