Skip to content

Commit

Permalink
staging:iio:light: Add some missing brackets to make sure code works …
Browse files Browse the repository at this point in the history
…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>
  • Loading branch information
Jonathan Cameron committed May 7, 2015
1 parent 2fdaf3f commit b91617e
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion drivers/staging/iio/light/isl29018.c
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down

0 comments on commit b91617e

Please sign in to comment.