Commit 4a737ec3 authored by Ioana Ciornei's avatar Ioana Ciornei Committed by Greg Kroah-Hartman

staging: iio: cdc: simplify return flow

Simplify return flow ad7150_write_event_params in order to reduce code length.
Since i2c_smbus_write_word_data only returns a negative value or 0 the change is safe.

This patch fixes the following coccicheck warning:

drivers/staging/iio/cdc/ad7150.c:182:2-5: WARNING: end returns can be simpified if negative or 0 value
Signed-off-by: default avatarIoana Ciornei <ciorneiioana@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent bd60ac14
...@@ -179,12 +179,9 @@ static int ad7150_write_event_params(struct iio_dev *indio_dev, ...@@ -179,12 +179,9 @@ static int ad7150_write_event_params(struct iio_dev *indio_dev,
/* Note completely different from the adaptive versions */ /* Note completely different from the adaptive versions */
case IIO_EV_TYPE_THRESH: case IIO_EV_TYPE_THRESH:
value = chip->threshold[rising][chan]; value = chip->threshold[rising][chan];
ret = i2c_smbus_write_word_data(chip->client, return i2c_smbus_write_word_data(chip->client,
ad7150_addresses[chan][3], ad7150_addresses[chan][3],
swab16(value)); swab16(value));
if (ret < 0)
return ret;
return 0;
case IIO_EV_TYPE_MAG_ADAPTIVE: case IIO_EV_TYPE_MAG_ADAPTIVE:
sens = chip->mag_sensitivity[rising][chan]; sens = chip->mag_sensitivity[rising][chan];
timeout = chip->mag_timeout[rising][chan]; timeout = chip->mag_timeout[rising][chan];
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment