Unverified Commit e699b2c8 authored by Pierre-Louis Bossart's avatar Pierre-Louis Bossart Committed by Mark Brown
Browse files

ASoC: rt5682: clarify expression 



cppcheck warning:

sound/soc/codecs/rt5682.c:2401:65: style: Boolean result is used in
bitwise operation. Clarify expression with
parentheses. [clarifyCondition]
      (pll_code.m_bp ? 0 : pll_code.m_code) << RT5682_PLL_M_SFT |

Signed-off-by: default avatarPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Link: https://lore.kernel.org/r/20210302212527.55158-23-pierre-louis.bossart@linux.intel.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 51cb94f6

sound/soc/codecs/rt5682.c

+3 −3
Original line number Diff line number Diff line
@@ -2396,10 +2396,10 @@ static int rt5682_set_component_pll(struct snd_soc_component *component, 
			pll_code.n_code, pll_code.k_code);



		snd_soc_component_write(component, RT5682_PLL_CTRL_1,

			pll_code.n_code << RT5682_PLL_N_SFT | pll_code.k_code);

			(pll_code.n_code << RT5682_PLL_N_SFT) | pll_code.k_code);

		snd_soc_component_write(component, RT5682_PLL_CTRL_2,

		    (pll_code.m_bp ? 0 : pll_code.m_code) << RT5682_PLL_M_SFT |

		    pll_code.m_bp << RT5682_PLL_M_BP_SFT | RT5682_PLL_RST);

			((pll_code.m_bp ? 0 : pll_code.m_code) << RT5682_PLL_M_SFT) |

			(pll_code.m_bp << RT5682_PLL_M_BP_SFT | RT5682_PLL_RST));

	}



	rt5682->pll_in[pll_id] = freq_in;