diff options
| author | Moksh Panicker <mokshpanicker.7@gmail.com> | 2026-07-02 11:46:49 +0000 |
|---|---|---|
| committer | Jonathan Cameron <jic23@kernel.org> | 2026-07-02 18:01:20 +0100 |
| commit | 220c2ad6f3ac4b32654404965991617c1cfe7861 (patch) | |
| tree | a04c310c3e86976bc144cade34a10f3a405157ba | |
| parent | 0790de31298adfb8f7c69235e3839c8a5f285dd7 (diff) | |
iio: adc: ad7779: Remove redundant dev_err_probe() after devm_request_irq()
devm_request_irq() already prints an error message on failure via the
IRQ core, so wrapping its return value in dev_err_probe() results in a
duplicate error message. Return the error directly instead.
Suggested-by: Joshua Crofts <joshua.crofts1@gmail.com>
Signed-off-by: Moksh Panicker <mokshpanicker.7@gmail.com>
Reviewed-by: Joshua Crofts <joshua.crofts1@gmail.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
| -rw-r--r-- | drivers/iio/adc/ad7779.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/iio/adc/ad7779.c b/drivers/iio/adc/ad7779.c index 5e2e1c31748b..87e0139c0f13 100644 --- a/drivers/iio/adc/ad7779.c +++ b/drivers/iio/adc/ad7779.c @@ -842,8 +842,7 @@ static int ad7779_setup_without_backend(struct ad7779_state *st, struct iio_dev IRQF_NO_THREAD | IRQF_NO_AUTOEN, indio_dev->name, st->trig); if (ret) - return dev_err_probe(dev, ret, "request IRQ %d failed\n", - st->spi->irq); + return ret; ret = devm_iio_trigger_register(dev, st->trig); if (ret) |