diff options
author | kbuild test robot <fengguang.wu@intel.com> | 2016-09-01 08:38:38 +0800 |
---|---|---|
committer | Jonathan Cameron <jic23@kernel.org> | 2016-09-03 17:49:53 +0100 |
commit | 2535cc7ae02bc1e4dc6b6d3771ba8d18cd824e10 (patch) | |
tree | ae948cef07a2a3f26b837ca84530186dca9c9372 | |
parent | 1b246fca4adaa0bf440b604366f2227cc4cde702 (diff) | |
download | linux-2535cc7ae02bc1e4dc6b6d3771ba8d18cd824e10.tar.bz2 |
iio: chemical: vz89x: fix boolreturn.cocci warnings
drivers/iio/chemical/vz89x.c:119:9-10: WARNING: return of 0/1 in function 'vz89x_measurement_is_valid' with return type bool
Return statements in functions returning bool should use
true/false instead of 1/0.
Generated by: scripts/coccinelle/misc/boolreturn.cocci
CC: Matt Ranostay <mranostay@gmail.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
-rw-r--r-- | drivers/iio/chemical/vz89x.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/iio/chemical/vz89x.c b/drivers/iio/chemical/vz89x.c index cd3870ead3fd..8e0e4415c161 100644 --- a/drivers/iio/chemical/vz89x.c +++ b/drivers/iio/chemical/vz89x.c @@ -160,7 +160,7 @@ static const struct attribute_group vz89x_attrs_group = { static bool vz89x_measurement_is_valid(struct vz89x_data *data) { if (data->buffer[VZ89X_VOC_SHORT_IDX] == 0) - return 1; + return true; return !!(data->buffer[data->chip->read_size - 1] > 0); } |