staging:iio:ad7298: Do not return error code in interrupt handler
The interrupt handler should only ever return one of the three irqreturn_t constants and not an error code. Also make sure to always call iio_trigger_notify_done before leaving the trigger handler. Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> Signed-off-by: Jonathan Cameron <jic23@kernel.org>
This commit is contained in:
parent
1764eff7d5
commit
abad65cbb2
|
@ -82,7 +82,7 @@ static irqreturn_t ad7298_trigger_handler(int irq, void *p)
|
||||||
|
|
||||||
b_sent = spi_sync(st->spi, &st->ring_msg);
|
b_sent = spi_sync(st->spi, &st->ring_msg);
|
||||||
if (b_sent)
|
if (b_sent)
|
||||||
return b_sent;
|
goto done;
|
||||||
|
|
||||||
if (indio_dev->scan_timestamp) {
|
if (indio_dev->scan_timestamp) {
|
||||||
time_ns = iio_get_time_ns();
|
time_ns = iio_get_time_ns();
|
||||||
|
@ -95,6 +95,8 @@ static irqreturn_t ad7298_trigger_handler(int irq, void *p)
|
||||||
buf[i] = be16_to_cpu(st->rx_buf[i]);
|
buf[i] = be16_to_cpu(st->rx_buf[i]);
|
||||||
|
|
||||||
indio_dev->buffer->access->store_to(ring, (u8 *)buf, time_ns);
|
indio_dev->buffer->access->store_to(ring, (u8 *)buf, time_ns);
|
||||||
|
|
||||||
|
done:
|
||||||
iio_trigger_notify_done(indio_dev->trig);
|
iio_trigger_notify_done(indio_dev->trig);
|
||||||
|
|
||||||
return IRQ_HANDLED;
|
return IRQ_HANDLED;
|
||||||
|
|
Loading…
Reference in New Issue