projects
/
linux-2.6-microblaze.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
63023e8
)
iio: proximity: sx9500: simplify code in write_event_config callback
author
Julien Stephan
<jstephan@baylibre.com>
Thu, 31 Oct 2024 15:27:00 +0000
(16:27 +0100)
committer
Jonathan 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
patch
|
blob
|
history
diff --git
a/drivers/iio/proximity/sx9500.c
b/drivers/iio/proximity/sx9500.c
index
3f4eace
..
e3da709
100644
(file)
--- a/
drivers/iio/proximity/sx9500.c
+++ b/
drivers/iio/proximity/sx9500.c
@@
-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);