Commit e4d8aef2 authored by Lars-Peter Clausen's avatar Lars-Peter Clausen Committed by Takashi Iwai

ALSA: usb: Use DIV_ROUND_UP() instead of open-coding it

Use DIV_ROUND_UP() instead of open-coding it. This documents intent
and 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) - 1) / (y))
+DIV_ROUND_UP(x, y)
// </smpl>
Signed-off-by: default avatarLars-Peter Clausen <lars@metafoo.de>
Link: https://lore.kernel.org/r/20201223172229.781-11-lars@metafoo.deSigned-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent a434713b
...@@ -254,7 +254,7 @@ static int get_relative_value(struct usb_mixer_elem_info *cval, int val) ...@@ -254,7 +254,7 @@ static int get_relative_value(struct usb_mixer_elem_info *cval, int val)
if (val < cval->min) if (val < cval->min)
return 0; return 0;
else if (val >= cval->max) else if (val >= cval->max)
return (cval->max - cval->min + cval->res - 1) / cval->res; return DIV_ROUND_UP(cval->max - cval->min, cval->res);
else else
return (val - cval->min) / cval->res; return (val - cval->min) / cval->res;
} }
...@@ -1338,7 +1338,7 @@ static int mixer_ctl_feature_info(struct snd_kcontrol *kcontrol, ...@@ -1338,7 +1338,7 @@ static int mixer_ctl_feature_info(struct snd_kcontrol *kcontrol,
} }
uinfo->value.integer.min = 0; uinfo->value.integer.min = 0;
uinfo->value.integer.max = uinfo->value.integer.max =
(cval->max - cval->min + cval->res - 1) / cval->res; DIV_ROUND_UP(cval->max - cval->min, cval->res);
} }
return 0; return 0;
} }
......
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