summaryrefslogtreecommitdiff
path: root/drivers/staging/iio
diff options
context:
space:
mode:
authorJonathan Cameron <jic23@kernel.org>2015-05-02 14:05:05 +0300
committerJonathan Cameron <jic23@kernel.org>2015-05-07 12:42:17 +0300
commitb91617ea62af5558af598ae9ad069db96f2ab00e (patch)
treeb30d358ddcfe6b630e18a923307865d25c27f7af /drivers/staging/iio
parent2fdaf3f4f8c718a5023db69e2d391d978e94703e (diff)
downloadlinux-b91617ea62af5558af598ae9ad069db96f2ab00e.tar.xz
staging:iio:light: Add some missing brackets to make sure code works as intended.
Note this is not a bug due to the fact the region cannot be reached without the sanity check passing. The autobuilder reported it as missaligned code which is kind of true as well. Signed-off-by: Jonathan Cameron <jic23@kernel.org> Cc: Roberta Dobrescu <roberta.dobrescu@gmail.com>
Diffstat (limited to 'drivers/staging/iio')
-rw-r--r--drivers/staging/iio/light/isl29018.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/staging/iio/light/isl29018.c b/drivers/staging/iio/light/isl29018.c
index 08ca9a4172e3..e646c5d24004 100644
--- a/drivers/staging/iio/light/isl29018.c
+++ b/drivers/staging/iio/light/isl29018.c
@@ -369,12 +369,13 @@ static int isl29018_write_raw(struct iio_dev *indio_dev,
}
break;
case IIO_CHAN_INFO_INT_TIME:
- if (chan->type == IIO_LIGHT)
+ if (chan->type == IIO_LIGHT) {
if (val != 0) {
mutex_unlock(&chip->lock);
return -EINVAL;
}
ret = isl29018_set_integration_time(chip, val2);
+ }
break;
case IIO_CHAN_INFO_SCALE:
if (chan->type == IIO_LIGHT)