On Sun, 13 Nov 2022 10:58:20 +0530 Deepak R Varma <drv@xxxxxxxxx> wrote: > Replace ternary operator by simple if based evaluation of the return > value. Issue identified using coccicheck. > > Signed-off-by: Deepak R Varma <drv@xxxxxxxxx> Applied to the togreg branch of iio.git and pushed out as testing to see if 0-day can find anything we missed. Thanks, Jonathan > --- > > Changes in v2: > 1. Use if based evaluation instead of using min macro > suggested by Joe Perches. > > > drivers/staging/iio/meter/ade7854-i2c.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/staging/iio/meter/ade7854-i2c.c b/drivers/staging/iio/meter/ade7854-i2c.c > index a9a06e8dda51..71b67dd3c8e9 100644 > --- a/drivers/staging/iio/meter/ade7854-i2c.c > +++ b/drivers/staging/iio/meter/ade7854-i2c.c > @@ -61,7 +61,10 @@ static int ade7854_i2c_write_reg(struct device *dev, > unlock: > mutex_unlock(&st->buf_lock); > > - return ret < 0 ? ret : 0; > + if (ret < 0) > + return ret; > + > + return 0; > } > > static int ade7854_i2c_read_reg(struct device *dev, > -- > 2.34.1 > > >
