From: Julien Stephan Date: Thu, 24 Oct 2024 09:11:26 +0000 (+0200) Subject: iio: light: veml6030: simplify code in write_event_config callback X-Git-Tag: v6.13-rc1~28^2~11^2~86 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=d567ff3603cf256e3434cf325302da5ddf56c69e;p=thirdparty%2Flinux.git iio: light: veml6030: simplify code in write_event_config callback 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 Link: https://patch.msgid.link/20241024-iio-fix-write-event-config-signature-v1-4-7d29e5a31b00@baylibre.com Signed-off-by: Jonathan Cameron --- diff --git a/drivers/iio/light/veml6030.c b/drivers/iio/light/veml6030.c index d6f3b104b0e6b..95751c1015909 100644 --- a/drivers/iio/light/veml6030.c +++ b/drivers/iio/light/veml6030.c @@ -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,