ASoC: rt1015: clarify expression
authorPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Tue, 2 Mar 2021 21:25:07 +0000 (15:25 -0600)
committerMark Brown <broonie@kernel.org>
Wed, 10 Mar 2021 13:09:56 +0000 (13:09 +0000)
cppcheck warning:

sound/soc/codecs/rt1015.c:894:61: style: Boolean result is used in
bitwise operation. Clarify expression with
parentheses. [clarifyCondition]
  (pll_code.m_bp ? 0 : pll_code.m_code) << RT1015_PLL_M_SFT |
                                                            ^

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20210302212527.55158-5-pierre-louis.bossart@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/rt1015.c

index 3cd967ae999d03a41e7ddcc755cd381b3d0da72a..10a7b7745f4be5b8e883d4f8df511a6e537c1930 100644 (file)
@@ -893,8 +893,9 @@ static int rt1015_set_component_pll(struct snd_soc_component *component,
                pll_code.n_code, pll_code.k_code);
 
        snd_soc_component_write(component, RT1015_PLL1,
-               (pll_code.m_bp ? 0 : pll_code.m_code) << RT1015_PLL_M_SFT |
-               pll_code.m_bp << RT1015_PLL_M_BP_SFT | pll_code.n_code);
+               ((pll_code.m_bp ? 0 : pll_code.m_code) << RT1015_PLL_M_SFT) |
+               (pll_code.m_bp << RT1015_PLL_M_BP_SFT) |
+               pll_code.n_code);
        snd_soc_component_write(component, RT1015_PLL2,
                pll_code.k_code);