diff options
| author | Crt Mori <[email protected]> | 2015-10-02 14:24:21 +0200 |
|---|---|---|
| committer | Jonathan Cameron <[email protected]> | 2015-10-03 10:15:25 +0100 |
| commit | 1de953e77b8c8b8cb6f734733cf20e95a1b0fbb8 (patch) | |
| tree | a482c774bd4097f91bff9a84f6aa061be16106ff /tools/perf/scripts/python/syscall-counts.py | |
| parent | c05dc2cce7422a5c05db470572b22fcd855f8cd7 (diff) | |
iio: mlx96014: Error checking from positive to negative
Dan Carpenter reported a static checker report and after his mail I
noticed that we actually return from function if positive value is
obtained from i2c read. This was remainder from when code was not in
separate function (which I changed during the review process).
Static checker reported
drivers/iio/temperature/mlx90614.c:167
mlx90614_iir_search()
warn: this cast is a no-op
which meant that cast before negating is useless. Dan also proposed a
solution on nicer bit operation form.
Also changed magic number to macro in process as that was confusing.
Reported-by: Dan Carpenter <[email protected]>
Signed-off-by: Crt Mori <[email protected]>
Signed-off-by: Jonathan Cameron <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/syscall-counts.py')
0 files changed, 0 insertions, 0 deletions