staging: iio: tsl2583: add error code to sysfs store functions

in_illuminance_input_target_store() and in_illuminance_calibrate_store()
validated the data from userspace, however it would not return an
error code to userspace if an invalid value was passed in. This patch
changes these functions so that they return -EINVAL if invalid data is
passed in.

Signed-off-by: Brian Masney <masneyb@onstation.org>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>

diff --git a/drivers/staging/iio/light/tsl2583.c b/drivers/staging/iio/light/tsl2583.c
index 14623740621d98d491bad60e3eda6c8fc8c32d32..98afa5bdfb29ce288bd63a94c630bc794e202e16 100644 (file)
--- a/drivers/staging/iio/light/tsl2583.c
+++ b/drivers/staging/iio/light/tsl2583.c
@@ -525,11 +525,10 @@ static ssize_t in_illuminance_input_target_store(struct device *dev,
        struct tsl2583_chip *chip = iio_priv(indio_dev);
        int value;
 
-       if (kstrtoint(buf, 0, &value))
+       if (kstrtoint(buf, 0, &value) || !value)
                return -EINVAL;
 
-       if (value)
-               chip->taos_settings.als_cal_target = value;
+       chip->taos_settings.als_cal_target = value;
 
        return len;
 }
@@ -541,11 +540,10 @@ static ssize_t in_illuminance_calibrate_store(struct device *dev,
        struct iio_dev *indio_dev = dev_to_iio_dev(dev);
        int value;
 
-       if (kstrtoint(buf, 0, &value))
+       if (kstrtoint(buf, 0, &value) || value != 1)
                return -EINVAL;
 
-       if (value == 1)
-               taos_als_calibrate(indio_dev);
+       taos_als_calibrate(indio_dev);
 
        return len;
 }