ASoC: rockchip: i2s: fixup clk div
authorSugar Zhang <sugar.zhang@rock-chips.com>
Mon, 5 Jun 2017 02:20:29 +0000 (10:20 +0800)
committerHuang, Tao <huangtao@rock-chips.com>
Tue, 6 Jun 2017 09:53:12 +0000 (17:53 +0800)
we found mclk maybe not precise as required because of PLL,
but it still can be used and no side effect. for example, if we
require mclk 11289600, but get 11289598, it doesn't matter.
so using DIV_ROUND_CLOSEST to fix it.

Change-Id: If8453a7a08b319da81b07d572b02247bd7e7bd27
Signed-off-by: Sugar Zhang <sugar.zhang@rock-chips.com>
sound/soc/rockchip/rockchip_i2s.c

index d7aa635e3693b684f32e1909d1bf5df7ed6fe52f..2c7e5ccca2dc2a27633e78bbb9b3606458aa9e9c 100644 (file)
@@ -282,10 +282,10 @@ static int rockchip_i2s_hw_params(struct snd_pcm_substream *substream,
        if (i2s->is_master_mode) {
                mclk_rate = clk_get_rate(i2s->mclk);
                bclk_rate = i2s->bclk_fs * params_rate(params);
-               if (bclk_rate && mclk_rate % bclk_rate)
+               if (!bclk_rate)
                        return -EINVAL;
 
-               div_bclk = mclk_rate / bclk_rate;
+               div_bclk = DIV_ROUND_CLOSEST(mclk_rate, bclk_rate);
                div_lrck = bclk_rate / params_rate(params);
                regmap_update_bits(i2s->regmap, I2S_CKR,
                                   I2S_CKR_MDIV_MASK,