[PATCH v4 5/8] iio: adc: ti-ads1015: drop unneeded < 0 error checks
Archit Anant <[email protected]>
| Newsgroups | org.kernel.vger.linux-iio,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
Several functions in the driver only return 0 on success or a negative error code on failure. Change the error checking to align with standard kernel coding style. Suggested-by: Andy Shevchenko <[email protected]> Signed-off-by: Archit Anant <[email protected]> --- drivers/iio/adc/ti-ads1015.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/iio/adc/ti-ads1015.c b/drivers/iio/adc/ti-ads1015.c index 0d1006041819..ba4a01850531 100644 --- a/drivers/iio/adc/ti-ads1015.c +++ b/drivers/iio/adc/ti-ads1015.c @@ -539,7 +539,7 @@ static int __ads1015_read_info_raw(struct ads1015_data *data, return -EBUSY; ret = ads1015_set_power_state(data, true); - if (ret < 0) + if (ret) return ret; ret = ads1015_get_adc_result(data, chan->address, val); @@ -727,7 +727,7 @@ static int ads1015_enable_event_config(struct ads1015_data *data, return ret; ret = ads1015_set_power_state(data, true); - if (ret < 0) + if (ret) return ret; ads1015_event_channel_enable(data, chan->address, comp_mode); @@ -994,7 +994,7 @@ static int ads1015_probe(struct i2c_client *client) ret = devm_iio_triggered_buffer_setup(dev, indio_dev, NULL, ads1015_trigger_handler, &ads1015_buffer_setup_ops); - if (ret < 0) + if (ret) return dev_err_probe(dev, ret, "iio triggered buffer setup failed\n"); -- 2.39.5