staging:iio:adt7410: Fix adt7410_set_mode return value
authorLars-Peter Clausen <lars@metafoo.de>
Mon, 22 Oct 2012 09:42:00 +0000 (10:42 +0100)
committerJonathan Cameron <jic23@kernel.org>
Sat, 17 Nov 2012 10:17:17 +0000 (10:17 +0000)
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>
drivers/staging/iio/adc/adt7410.c

index 5e93d868b3feffd872c810431d6b4ff0a6067b2d..6493cd0a48dd9a425cd990bd12c006926da076a6 100644 (file)
@@ -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,