diff options
| author |   Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> | 2021-03-10 18:43:28 -0600 |
|---|---|---|
| committer |   Mark Brown <broonie@kernel.org> | 2021-03-11 13:24:15 +0000 |
| commit | 729d42a4ab1a7bc9f4b5c37c57a3a4270333351f (patch) | |
| tree | df5289ac6df5bd00ef0dbfd7978f8434b3d3c487 | |
| parent | ASoC: wm8958-dsp2: rename local 'control' arrays (diff) | |
| download | linux-dev-729d42a4ab1a7bc9f4b5c37c57a3a4270333351f.tar.xz linux-dev-729d42a4ab1a7bc9f4b5c37c57a3a4270333351f.zip | |
ASoC: wm8978: clarify expression
cppcheck warning:
sound/soc/codecs/wm8978.c:727:57: style: Clarify calculation
precedence for '&' and '?'. [clarifyCalculation]
enum wm8978_sysclk_src current_clk_id = clking & 0x100 ?
^
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20210311004332.120901-7-pierre-louis.bossart@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
| -rw-r--r-- | sound/soc/codecs/wm8978.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/codecs/wm8978.c b/sound/soc/codecs/wm8978.c index 4b5ecd142249..7091e1a9d516 100644 --- a/sound/soc/codecs/wm8978.c +++ b/sound/soc/codecs/wm8978.c @@ -724,7 +724,7 @@ static int wm8978_hw_params(struct snd_pcm_substream *substream, /* Sampling rate mask = 0xe (for filters) */ u16 add_ctl = snd_soc_component_read(component, WM8978_ADDITIONAL_CONTROL) & ~0xe; u16 clking = snd_soc_component_read(component, WM8978_CLOCKING); - enum wm8978_sysclk_src current_clk_id = clking & 0x100 ? + enum wm8978_sysclk_src current_clk_id = (clking & 0x100) ? WM8978_PLL : WM8978_MCLK; unsigned int f_sel, diff, diff_best = INT_MAX; int i, best = 0; |