Commit e8f45e33 authored Oct 22, 2012 by Lars-Peter Clausen Committed by Jonathan Cameron Nov 17, 2012

staging:iio:adt7410: Fix adt7410_set_mode return value

The function is expected to return the number of bytes consumed and as long as
not all bytes have been consumed the function will be called again. Currently
the function returns 'ret', which will always be 0 in this case, so we end up in
a endless loop since the caller will assume that no bytes have been consumed. So
instead return len as it is supposed to.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>

parent 69bcd3bf

drivers/staging/iio/adc/adt7410.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -181,7 +181,7 @@ static ssize_t adt7410_store_mode(struct device *dev,

		chip->config = config;

		return ret;
		return len;
		}

		static IIO_DEVICE_ATTR(mode, S_IRUGO \| S_IWUSR,