]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
iio: light: veml6030: simplify code in write_event_config callback
authorJulien Stephan <jstephan@baylibre.com>
Thu, 24 Oct 2024 09:11:26 +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-4-7d29e5a31b00@baylibre.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/light/veml6030.c

index d6f3b104b0e6be8e095a1ad9f20acb81ce492630..95751c1015909cd80543a559673f435d1a168f68 100644 (file)
@@ -826,9 +826,6 @@ static int veml6030_write_interrupt_config(struct iio_dev *indio_dev,
        int ret;
        struct veml6030_data *data = iio_priv(indio_dev);
 
-       if (state < 0 || state > 1)
-               return -EINVAL;
-
        ret = veml6030_als_shut_down(data);
        if (ret < 0) {
                dev_err(&data->client->dev,