projects / openwrt / staging / blogic.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 9a5094c) | patch
iio: light: si1133 fix test for val being not zero or not one.
authorColin Ian King <colin.king@canonical.com>
Mon, 30 Jul 2018 12:59:18 +0000 (13:59 +0100)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Mon, 30 Jul 2018 18:28:47 +0000 (19:28 +0100)
commit1e96e93a3569fd7a4d3199d9d912345ad3b8ce4c
tree8a709016982e48dffcb072412e835311cedfa4aftree | snapshot
parent9a5094ca29ea9b1da301b31fd377c0c0c4c23034commit | diff
iio: light: si1133 fix test for val being not zero or not one.

The current check on val always results in true and so the
call to sii1133_update_adcsens never gets called. Fix this check
so it returns with -EINVAL only when val is not zero and not one.

Detected by CoverityScan, CID#1472099 ("Logically dead code")

Fixes: e01e7eaf37d8 ("iio: light: introduce si1133")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/light/si1133.c diff | blob | history
John Crispins staging tree