Commit d877539a authored by Nuno Sa's avatar Nuno Sa Committed by Jonathan Cameron
Browse files

iio: adis_buffer: do not return ints in irq handlers 



On an IRQ handler we should not return normal error codes as 'irqreturn_t'
is expected.

Not necessarily stable material as the old check cannot fail, so it's a bug
we can not hit.

Fixes: ccd2b52f ("staging:iio: Add common ADIS library")
Reviewed-by: default avatarAlexandru Ardelean <ardeleanalex@gmail.com>
Signed-off-by: default avatarNuno Sa <nuno.sa@analog.com>
Link: https://lore.kernel.org/r/20210422101911.135630-2-nuno.sa@analog.com


Signed-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
parent 49418666

drivers/iio/imu/adis_buffer.c

+0 −3
Original line number Diff line number Diff line
@@ -129,9 +129,6 @@ static irqreturn_t adis_trigger_handler(int irq, void *p) 
	struct adis *adis = iio_device_get_drvdata(indio_dev);

	int ret;



	if (!adis->buffer)

		return -ENOMEM;



	if (adis->data->has_paging) {

		mutex_lock(&adis->state_lock);

		if (adis->current_page != 0) {