ASoC: cs35l36: Use DIV_ROUND_UP() instead of open-coding it
authorShang XiaoJing <shangxiaojing@huawei.com>
Tue, 27 Sep 2022 14:09:45 +0000 (22:09 +0800)
committerMark Brown <broonie@kernel.org>
Mon, 17 Oct 2022 11:48:20 +0000 (12:48 +0100)
Use DIV_ROUND_UP() instead of open-coding it, which intents and makes it
more clear what is going on for the casual reviewer.

The Coccinelle references Commit e4d8aef21403 ("ALSA: usb: Use
DIV_ROUND_UP() instead of open-coding it").

Signed-off-by: Shang XiaoJing <shangxiaojing@huawei.com>
Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20220927140948.17696-2-shangxiaojing@huawei.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/cs35l36.c

index 31ae752e242f89189c93ad690bd0ffeadaef208d..a078dd422ea18fb2e3e3e0bfd5cfa8fd6809cc72 100644 (file)
@@ -918,8 +918,8 @@ static int cs35l36_dai_set_sysclk(struct snd_soc_dai *dai, int clk_id,
                fs1 = CS35L36_FS1_DEFAULT_VAL;
                fs2 = CS35L36_FS2_DEFAULT_VAL;
        } else {
-               fs1 = 3 * ((CS35L36_FS_NOM_6MHZ * 4 + freq - 1) / freq) + 4;
-               fs2 = 5 * ((CS35L36_FS_NOM_6MHZ * 4 + freq - 1) / freq) + 4;
+               fs1 = 3 * DIV_ROUND_UP(CS35L36_FS_NOM_6MHZ * 4, freq) + 4;
+               fs2 = 5 * DIV_ROUND_UP(CS35L36_FS_NOM_6MHZ * 4, freq) + 4;
        }
 
        regmap_write(cs35l36->regmap, CS35L36_TESTKEY_CTRL,