iio: light: ltr501: simplify code in write_event_config callback
authorJulien Stephan <jstephan@baylibre.com>
Thu, 24 Oct 2024 09:11:25 +0000 (11:11 +0200)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Fri, 1 Nov 2024 14:54:45 +0000 (14:54 +0000)
iio_ev_state_store is actually using kstrtobool to check user
input, then gives the converted boolean value to the write_event_config
callback.

Remove useless code in write_event_config callback.

Signed-off-by: Julien Stephan <jstephan@baylibre.com>
Link: https://patch.msgid.link/20241024-iio-fix-write-event-config-signature-v1-3-7d29e5a31b00@baylibre.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/light/ltr501.c

index 469e058..616dc69 100644 (file)
@@ -1082,10 +1082,6 @@ static int ltr501_write_event_config(struct iio_dev *indio_dev,
        struct ltr501_data *data = iio_priv(indio_dev);
        int ret;
 
-       /* only 1 and 0 are valid inputs */
-       if (state != 1  && state != 0)
-               return -EINVAL;
-
        switch (chan->type) {
        case IIO_INTENSITY:
                mutex_lock(&data->lock_als);