Debugged improper setting of PLL fields in WM8580 driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



   Bug was caught while trying to use WM8580 as I2S master on SMDK.
   Symptoms were lesser LRCLK read by CRO(41.02 instead of 44.1 KHz)
   Solved by referring to WM8580A manual and setting mask value correctly
   and making the code to not touch 'reserved' bits of PLL4 register.

   Signed-off-by: Jassi <jassi.brar@xxxxxxxxxxx>


diff --git a/sound/soc/codecs/wm8580.c b/sound/soc/codecs/wm8580.c
index d547347..6bded8c 100644
--- a/sound/soc/codecs/wm8580.c
+++ b/sound/soc/codecs/wm8580.c
@@ -458,12 +458,12 @@ static int wm8580_set_dai_pll(struct snd_soc_dai
*codec_dai,
 		return 0;

 	snd_soc_write(codec, WM8580_PLLA1 + offset, pll_div.k & 0x1ff);
-	snd_soc_write(codec, WM8580_PLLA2 + offset, (pll_div.k >> 9) & 0xff);
+	snd_soc_write(codec, WM8580_PLLA2 + offset, (pll_div.k >> 9) & 0x1ff);
 	snd_soc_write(codec, WM8580_PLLA3 + offset,
 		     (pll_div.k >> 18 & 0xf) | (pll_div.n << 4));

 	reg = snd_soc_read(codec, WM8580_PLLA4 + offset);
-	reg &= ~0x3f;
+	reg &= ~0x1b;
 	reg |= pll_div.prescale | pll_div.postscale << 1 |
 		pll_div.freqmode << 3;
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux