ASoC: wm8510 pll settings

When setting WM8510_MCLKDIV the pll was turned off.

When setting pll frequency you got twice the expected freq, because
the  code calculated  with postscaler of 8,  but  the hardware divide by 4.

Signed-off-by: Jonas Andersson <jonas@microbit.se>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>

2 files changed, 14 insertions, 14 deletions

diff --git a/sound/soc/atmel/playpaq_wm8510.c b/sound/soc/atmel/playpaq_wm8510.c
index 43dd8cee83c6..70657534e6b1 100644
--- a/sound/soc/atmel/playpaq_wm8510.c
+++ b/sound/soc/atmel/playpaq_wm8510.c
@@ -164,38 +164,38 @@ static int playpaq_wm8510_hw_params(struct snd_pcm_substream *substream,
          */
         switch (params_rate(params)) {
         case 48000:
-                pll_out = 12288000;
-                mclk_div = WM8510_MCLKDIV_1;
+                pll_out = 24576000;
+                mclk_div = WM8510_MCLKDIV_2;
                 bclk = WM8510_BCLKDIV_8;
                 break;
 
         case 44100:
-                pll_out = 11289600;
-                mclk_div = WM8510_MCLKDIV_1;
+                pll_out = 22579200;
+                mclk_div = WM8510_MCLKDIV_2;
                 bclk = WM8510_BCLKDIV_8;
                 break;
 
         case 22050:
-                pll_out = 11289600;
-                mclk_div = WM8510_MCLKDIV_2;
+                pll_out = 22579200;
+                mclk_div = WM8510_MCLKDIV_4;
                 bclk = WM8510_BCLKDIV_8;
                 break;
 
         case 16000:
-                pll_out = 12288000;
-                mclk_div = WM8510_MCLKDIV_3;
+                pll_out = 24576000;
+                mclk_div = WM8510_MCLKDIV_6;
                 bclk = WM8510_BCLKDIV_8;
                 break;
 
         case 11025:
-                pll_out = 11289600;
-                mclk_div = WM8510_MCLKDIV_4;
+                pll_out = 22579200;
+                mclk_div = WM8510_MCLKDIV_8;
                 bclk = WM8510_BCLKDIV_8;
                 break;
 
         case 8000:
-                pll_out = 12288000;
-                mclk_div = WM8510_MCLKDIV_6;
+                pll_out = 24576000;
+                mclk_div = WM8510_MCLKDIV_12;
                 bclk = WM8510_BCLKDIV_8;
                 break;
 
diff --git a/sound/soc/codecs/wm8510.c b/sound/soc/codecs/wm8510.c
index f01078cfbd72..6d4ef71e9195 100644
--- a/sound/soc/codecs/wm8510.c
+++ b/sound/soc/codecs/wm8510.c
@@ -336,7 +336,7 @@ static int wm8510_set_dai_pll(struct snd_soc_dai *codec_dai,
                 return 0;
         }
 
-        pll_factors(freq_out*8, freq_in);
+        pll_factors(freq_out*4, freq_in);
 
         wm8510_write(codec, WM8510_PLLN, (pll_div.pre_div << 4) | pll_div.n);
         wm8510_write(codec, WM8510_PLLK1, pll_div.k >> 18);
@@ -367,7 +367,7 @@ static int wm8510_set_dai_clkdiv(struct snd_soc_dai *codec_dai,
                 wm8510_write(codec, WM8510_GPIO, reg | div);
                 break;
         case WM8510_MCLKDIV:
-                reg = wm8510_read_reg_cache(codec, WM8510_CLOCK) & 0x1f;
+                reg = wm8510_read_reg_cache(codec, WM8510_CLOCK) & 0x11f;
                 wm8510_write(codec, WM8510_CLOCK, reg | div);
                 break;
         case WM8510_ADCCLK: