diff options
author | Lars-Peter Clausen <lars@metafoo.de> | 2020-12-23 18:22:18 +0100 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2020-12-25 09:11:50 +0100 |
commit | 7095be6d2cc2870299c9e6c6c2df3148be410c4a (patch) | |
tree | d190155841a52b6296f556eebad8f57bb4d392a8 /sound/drivers | |
parent | 7ca4282ade77de53b6e9ffa2695566e5d35dab1e (diff) | |
download | lwn-7095be6d2cc2870299c9e6c6c2df3148be410c4a.tar.gz lwn-7095be6d2cc2870299c9e6c6c2df3148be410c4a.zip |
ALSA: dummy: 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: Lars-Peter Clausen <lars@metafoo.de>
Link: https://lore.kernel.org/r/20201223172229.781-7-lars@metafoo.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/drivers')
-rw-r--r-- | sound/drivers/dummy.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/drivers/dummy.c b/sound/drivers/dummy.c index b5486de08b97..316c9afadefe 100644 --- a/sound/drivers/dummy.c +++ b/sound/drivers/dummy.c @@ -236,7 +236,7 @@ struct dummy_systimer_pcm { static void dummy_systimer_rearm(struct dummy_systimer_pcm *dpcm) { mod_timer(&dpcm->timer, jiffies + - (dpcm->frac_period_rest + dpcm->rate - 1) / dpcm->rate); + DIV_ROUND_UP(dpcm->frac_period_rest, dpcm->rate)); } static void dummy_systimer_update(struct dummy_systimer_pcm *dpcm) |