diff options
| author | Jonathan Cameron <jic23@kernel.org> | 2015-05-02 12:05:05 +0100 |
|---|---|---|
| committer | Jonathan Cameron <jic23@kernel.org> | 2015-05-07 10:42:17 +0100 |
| commit | b91617ea62af5558af598ae9ad069db96f2ab00e (patch) | |
| tree | b30d358ddcfe6b630e18a923307865d25c27f7af /drivers/staging/iio | |
| parent | 2fdaf3f4f8c718a5023db69e2d391d978e94703e (diff) | |
| download | op-kernel-dev-b91617ea62af5558af598ae9ad069db96f2ab00e.zip op-kernel-dev-b91617ea62af5558af598ae9ad069db96f2ab00e.tar.gz | |
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.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/staging/iio/light/isl29018.c b/drivers/staging/iio/light/isl29018.c index 08ca9a4..e646c5d 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) |
