From: Julien Stephan Date: Thu, 24 Oct 2024 09:11:25 +0000 (+0200) Subject: iio: light: ltr501: simplify code in write_event_config callback X-Git-Tag: v6.13-rc1~28^2~11^2~87 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=c9fd4cc90c0f8fb006797a59fecdcd69ce7211e3;p=thirdparty%2Flinux.git iio: light: ltr501: 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-3-7d29e5a31b00@baylibre.com Signed-off-by: Jonathan Cameron --- diff --git a/drivers/iio/light/ltr501.c b/drivers/iio/light/ltr501.c index 469e05866befb..616dc6921702a 100644 --- a/drivers/iio/light/ltr501.c +++ b/drivers/iio/light/ltr501.c @@ -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);