diff options
author | Peter Meerwald <pmeerw@pmeerw.net> | 2015-06-14 17:09:35 -0400 |
---|---|---|
committer | Jonathan Cameron <jic23@kernel.org> | 2015-06-21 09:54:30 -0400 |
commit | 33361e5678a541f82f29f85467d589e7bf8da76b (patch) | |
tree | d20dc9022102813333fe06a213bbe3d8768cd9c9 | |
parent | 7a1d0d91c94305fa5802a53df3a54c0ea1963c48 (diff) |
iio: light: tcs3414: Fix bug preventing to set integration time
the millisecond values in tcs3414_times should be checked against
val2, not val, which is always zero.
Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>
Reported-by: Stephan Kleisinger <stephan.kleisinger@gmail.com>
Cc: <Stable@vger.kernel.org>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
-rw-r--r-- | drivers/iio/light/tcs3414.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/iio/light/tcs3414.c b/drivers/iio/light/tcs3414.c index 71c2bde275aa..f8b1df018abe 100644 --- a/drivers/iio/light/tcs3414.c +++ b/drivers/iio/light/tcs3414.c | |||
@@ -185,7 +185,7 @@ static int tcs3414_write_raw(struct iio_dev *indio_dev, | |||
185 | if (val != 0) | 185 | if (val != 0) |
186 | return -EINVAL; | 186 | return -EINVAL; |
187 | for (i = 0; i < ARRAY_SIZE(tcs3414_times); i++) { | 187 | for (i = 0; i < ARRAY_SIZE(tcs3414_times); i++) { |
188 | if (val == tcs3414_times[i] * 1000) { | 188 | if (val2 == tcs3414_times[i] * 1000) { |
189 | data->timing &= ~TCS3414_INTEG_MASK; | 189 | data->timing &= ~TCS3414_INTEG_MASK; |
190 | data->timing |= i; | 190 | data->timing |= i; |
191 | return i2c_smbus_write_byte_data( | 191 | return i2c_smbus_write_byte_data( |