diff options
author | Lars-Peter Clausen <lars@metafoo.de> | 2020-12-23 18:22:25 +0100 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2020-12-25 09:11:55 +0100 |
commit | 4743feb9f43b896b22af93e20307eb62fa1299d3 (patch) | |
tree | 95d07e58bab30728e0b18d45c9aa2d3b46741460 | |
parent | ALSA: oss: Use DIV_ROUND_CLOSEST() instead of open-coding it (diff) | |
download | linux-4743feb9f43b896b22af93e20307eb62fa1299d3.tar.xz linux-4743feb9f43b896b22af93e20307eb62fa1299d3.zip |
ALSA: sonicvibes: Use DIV_ROUND_CLOSEST() instead of open-coding it
Use DIV_ROUND_CLOSEST() 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) / 2)) / (y)
+DIV_ROUND_CLOSEST(x, y)
// </smpl>
Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Link: https://lore.kernel.org/r/20201223172229.781-14-lars@metafoo.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
-rw-r--r-- | sound/pci/sonicvibes.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/pci/sonicvibes.c b/sound/pci/sonicvibes.c index ecdd54d7a4e1..bd805e416e12 100644 --- a/sound/pci/sonicvibes.c +++ b/sound/pci/sonicvibes.c @@ -570,7 +570,7 @@ static void snd_sonicvibes_set_dac_rate(struct sonicvibes * sonic, unsigned int unsigned int div; unsigned long flags; - div = (rate * 65536 + SV_FULLRATE / 2) / SV_FULLRATE; + div = DIV_ROUND_CLOSEST(rate * 65536, SV_FULLRATE); if (div > 65535) div = 65535; spin_lock_irqsave(&sonic->reg_lock, flags); |