iio: proximity: sx9500: simplify code in write_event_config callback
authorJulien Stephan <jstephan@baylibre.com>
Thu, 31 Oct 2024 15:27:00 +0000 (16:27 +0100)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Sun, 3 Nov 2024 20:33:43 +0000 (20:33 +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/20241031-iio-fix-write-event-config-signature-v2-5-2bcacbb517a2@baylibre.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/proximity/sx9500.c

index 3f4eace..e3da709 100644 (file)
@@ -551,7 +551,7 @@ static int sx9500_write_event_config(struct iio_dev *indio_dev,
 
        mutex_lock(&data->mutex);
 
-       if (state == 1) {
+       if (state) {
                ret = sx9500_inc_chan_users(data, chan->channel);
                if (ret < 0)
                        goto out_unlock;
@@ -571,7 +571,7 @@ static int sx9500_write_event_config(struct iio_dev *indio_dev,
        goto out_unlock;
 
 out_undo_chan:
-       if (state == 1)
+       if (state)
                sx9500_dec_chan_users(data, chan->channel);
        else
                sx9500_inc_chan_users(data, chan->channel);