diff options
| author | Nikhil Gautam <nikhilgtr@gmail.com> | 2026-07-14 17:01:33 +0530 |
|---|---|---|
| committer | Jonathan Cameron <jonathan.cameron@oss.qualcomm.com> | 2026-08-07 23:51:04 +0100 |
| commit | d0f21621f8b2b46661ea066d20705dbf7253db87 (patch) | |
| tree | 411b9b0f940b378d3ac8d2bc4d205cfffd02bb23 | |
| parent | afa28741c9a2cf6edb2e41e25ff146a562160bb3 (diff) | |
| download | linux-d0f21621f8b2b46661ea066d20705dbf7253db87.tar.gz linux-d0f21621f8b2b46661ea066d20705dbf7253db87.zip | |
iio: light: opt4001: Reject integration times with a non-zero seconds part
opt4001_write_raw() only looks at val2 when setting the integration
time, so a write such as 1.000600 is silently accepted as 600 us.
Return -EINVAL if val is non-zero.
Fixes: 9a9608418292 ("iio: light: Add support for TI OPT4001 light sensor")
Signed-off-by: Nikhil Gautam <nikhilgtr@gmail.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@intel.com>
Cc: <Stable@vger.kernel.org>
Signed-off-by: Jonathan Cameron <jonathan.cameron@oss.qualcomm.com>
| -rw-r--r-- | drivers/iio/light/opt4001.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/iio/light/opt4001.c b/drivers/iio/light/opt4001.c index 944923cb8f28..2448d6330489 100644 --- a/drivers/iio/light/opt4001.c +++ b/drivers/iio/light/opt4001.c @@ -269,6 +269,9 @@ static int opt4001_write_raw(struct iio_dev *indio_dev, switch (mask) { case IIO_CHAN_INFO_INT_TIME: + if (val) + return -EINVAL; + int_time = opt4001_als_time_to_index(val2); if (int_time < 0) return int_time; |