staging:iio:ad7606: Remove out-of-band error reporting
authorLars-Peter Clausen <lars@metafoo.de>
Wed, 19 Oct 2016 17:06:59 +0000 (19:06 +0200)
committerJonathan Cameron <jic23@kernel.org>
Sun, 23 Oct 2016 18:34:13 +0000 (19:34 +0100)
Currently the ad7606 driver prints a error message to the kernel log when
an application writes an invalid value to a sysfs attribute. While for
initial driver development and testing this might be useful it is quite
disadvantageous in a production environment. The write() call to the sysfs
attribute will already return an error if the value was invalid so the
application is aware that the operation failed. And generally speaking it
is impossible for an application to reliably match a log message in the
kernel log to a specific operation it performed, so the message becomes
just noise and might distract from more critical messages.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/staging/iio/adc/ad7606_core.c

index a16c6f5a333bde9f9719855c35c8013368e63ceb..d3a35512261a8d052f4ae772781acf3d07b4feb5 100644 (file)
@@ -132,10 +132,9 @@ static ssize_t ad7606_store_range(struct device *dev,
        if (ret)
                return ret;
 
-       if (!(lval == 5000 || lval == 10000)) {
-               dev_err(dev, "range is not supported\n");
+       if (!(lval == 5000 || lval == 10000))
                return -EINVAL;
-       }
+
        mutex_lock(&indio_dev->mlock);
        gpio_set_value(st->pdata->gpio_range, lval == 10000);
        st->range = lval;
@@ -174,11 +173,8 @@ static int ad7606_write_raw(struct iio_dev *indio_dev,
                if (val2)
                        return -EINVAL;
                ret = ad7606_oversampling_get_index(val);
-               if (ret < 0) {
-                       dev_err(st->dev, "oversampling %d is not supported\n",
-                               val);
+               if (ret < 0)
                        return ret;
-               }
 
                mutex_lock(&indio_dev->mlock);
                gpio_set_value(st->pdata->gpio_os0, (ret >> 0) & 1);