diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2015-08-18 12:16:33 +0300 |
|---|---|---|
| committer | Jonathan Cameron <jic23@kernel.org> | 2015-08-31 16:29:52 +0100 |
| commit | 7d87b3c5c8602df4ce6d615b33ae6dc15438fa29 (patch) | |
| tree | bef0bea1b20eaf2bb7818db6f0ec3443bb4b409f /drivers/iio/light/ltr501.c | |
| parent | aff268cd532e99ced3c8f48d01118912eb002bbf (diff) | |
| download | op-kernel-dev-7d87b3c5c8602df4ce6d615b33ae6dc15438fa29.zip op-kernel-dev-7d87b3c5c8602df4ce6d615b33ae6dc15438fa29.tar.gz | |
iio: tsl4531: fix error handling in tsl4531_check_id()
The tsl4531_check_id() function returned 1 on "found" and 0 on "not
found" and negative error codes on failure. This was non-standard and
bug prone. The caller treated all non-zero values including error codes
as "found".
This patch fixes it by changing the tsl4531_check_id() to return zero on
success or a negative error code, and updates the caller.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Peter Meerwald <pmeerw@pmeerw.net>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Diffstat (limited to 'drivers/iio/light/ltr501.c')
0 files changed, 0 insertions, 0 deletions
