Quoting Daniel Campello (2020-07-28 16:05:06) > The first patch resends the DT binding for the driver that was merged in > v5.8-rc1 with a small change to update for proper regulators. The second > through the eleventh patch fixes several issues dropped from v8 to v9 > when the initial patch was merged. The twelveth patch fixes a few > printks that are missing newlines and should be totally non-trivial to > apply. The thirteenth patch drops channel_users because it's unused. The > final patch adds support to enable the svdd and vdd supplies so that > this driver can work on a board where the svdd supply isn't enabled at > boot and needs to be turned on before this driver starts to communicate > with the chip. > Can you add this patch onto the end? ----8<---- From: Stephen Boyd <swboyd@xxxxxxxxxxxx> Subject: [PATCH] iio: sx9310: Use irq trigger flags from firmware We shouldn't need to set default irq trigger flags here as the firmware should have properly indicated the trigger type, i.e. level low, in the DT or ACPI tables. Signed-off-by: Stephen Boyd <swboyd@xxxxxxxxxxxx> --- drivers/iio/proximity/sx9310.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/iio/proximity/sx9310.c b/drivers/iio/proximity/sx9310.c index 24a2360b6314..2106b9141928 100644 --- a/drivers/iio/proximity/sx9310.c +++ b/drivers/iio/proximity/sx9310.c @@ -945,7 +945,7 @@ static int sx9310_probe(struct i2c_client *client) ret = devm_request_threaded_irq(dev, client->irq, sx9310_irq_handler, sx9310_irq_thread_handler, - IRQF_TRIGGER_LOW | IRQF_ONESHOT, + IRQF_ONESHOT, "sx9310_event", indio_dev); if (ret) return ret; -- Sent by a computer, using git, on the internet
