ASoC: cs35l34: Simplify the logic to set CS35L34_MCLK_CTL setting
authorAxel Lin <axel.lin@ingics.com>
Sat, 3 Dec 2016 07:10:50 +0000 (15:10 +0800)
committerMark Brown <broonie@kernel.org>
Tue, 6 Dec 2016 12:22:28 +0000 (12:22 +0000)
The logic of "value = ~CS35L34_MCLK_DIV & CS35L34_MCLK_RATE_XXXXXX;" is
unnecessary complex. By setting CS35L34_MCLK_DIV | CS35L34_MCLK_RATE_MASK
as the mask for regmap_update_bits() call, what the code does is exactly
the same as setting value = CS35L34_MCLK_RATE_XXXXXX.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
Acked-by: Paul Handrigan <Paul.Handrigan@cirrus.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/cs35l34.c

index e0f672af1d847aa534b2043b2870583f65c73ab6..7c5d1510cf2cb293638ce3f7ef360cd5f7789b1c 100644 (file)
@@ -610,15 +610,15 @@ static int cs35l34_dai_set_sysclk(struct snd_soc_dai *dai,
 
        switch (freq) {
        case CS35L34_MCLK_5644:
-               value = ~CS35L34_MCLK_DIV & CS35L34_MCLK_RATE_5P6448;
+               value = CS35L34_MCLK_RATE_5P6448;
                cs35l34->mclk_int = freq;
        break;
        case CS35L34_MCLK_6:
-               value = ~CS35L34_MCLK_DIV & CS35L34_MCLK_RATE_6P0000;
+               value = CS35L34_MCLK_RATE_6P0000;
                cs35l34->mclk_int = freq;
        break;
        case CS35L34_MCLK_6144:
-               value = ~CS35L34_MCLK_DIV & CS35L34_MCLK_RATE_6P1440;
+               value = CS35L34_MCLK_RATE_6P1440;
                cs35l34->mclk_int = freq;
        break;
        case CS35L34_MCLK_11289: