Skip to content

Commit 6c20c80

Browse files
Dimitris Papastamosbroonie
authored andcommitted
ASoC: WM8804: Power down the PLL correctly
The PLL is disabled when the corresponding bit is set not the other way around. This commit depends on my other commit with Subject "ASoC: WM8804: Refactor set_pll code to avoid GCC warnings". Signed-off-by: Dimitris Papastamos <[email protected]> Acked-by: Liam Girdwood <[email protected]> Signed-off-by: Mark Brown <[email protected]>
1 parent 86ce6c9 commit 6c20c80

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

sound/soc/codecs/wm8804.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -395,7 +395,7 @@ static int wm8804_set_pll(struct snd_soc_dai *dai, int pll_id,
395395
codec = dai->codec;
396396
if (!freq_in || !freq_out) {
397397
/* disable the PLL */
398-
snd_soc_update_bits(codec, WM8804_PWRDN, 0x1, 0);
398+
snd_soc_update_bits(codec, WM8804_PWRDN, 0x1, 0x1);
399399
return 0;
400400
} else {
401401
int ret;
@@ -406,7 +406,7 @@ static int wm8804_set_pll(struct snd_soc_dai *dai, int pll_id,
406406
return ret;
407407

408408
/* power down the PLL before reprogramming it */
409-
snd_soc_update_bits(codec, WM8804_PWRDN, 0x1, 0);
409+
snd_soc_update_bits(codec, WM8804_PWRDN, 0x1, 0x1);
410410

411411
if (!freq_in || !freq_out)
412412
return 0;
@@ -423,7 +423,7 @@ static int wm8804_set_pll(struct snd_soc_dai *dai, int pll_id,
423423
snd_soc_write(codec, WM8804_PLL3, pll_div.k >> 16);
424424

425425
/* power up the PLL */
426-
snd_soc_update_bits(codec, WM8804_PWRDN, 0x1, 0x1);
426+
snd_soc_update_bits(codec, WM8804_PWRDN, 0x1, 0);
427427
}
428428

429429
return 0;

0 commit comments

Comments
 (0)