On Wed, 23 Dec 2020 at 03:16, Lars-Peter Clausen <lars@xxxxxxxxxx> wrote: > > Use DIV_ROUND_CLOSEST() instead of open-coding it. This makes it more clear > what is going on for the casual reviewer. > > Generated using the following the Coccinelle semantic patch. > > // <smpl> > @@ > expression x, y; > @@ > -((x) + ((y) / 2)) / (y) > +DIV_ROUND_CLOSEST(x, y) > // </smpl> > > Signed-off-by: Lars-Peter Clausen <lars@xxxxxxxxxx> Acked-by: Chunyan Zhang <zhang.lyra@xxxxxxxxx> Thanks, Chunyan > --- > drivers/iio/adc/sc27xx_adc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/iio/adc/sc27xx_adc.c b/drivers/iio/adc/sc27xx_adc.c > index aa32a1f385e2..301cf66de695 100644 > --- a/drivers/iio/adc/sc27xx_adc.c > +++ b/drivers/iio/adc/sc27xx_adc.c > @@ -307,7 +307,7 @@ static int sc27xx_adc_convert_volt(struct sc27xx_adc_data *data, int channel, > > sc27xx_adc_volt_ratio(data, channel, scale, &numerator, &denominator); > > - return (volt * denominator + numerator / 2) / numerator; > + return DIV_ROUND_CLOSEST(volt * denominator, numerator); > } > > static int sc27xx_adc_read_processed(struct sc27xx_adc_data *data, > -- > 2.20.1 >
