ASoC: wm8962: Don't automatically enable and disable FLL
authorMark Brown <broonie@opensource.wolfsonmicro.com>
Fri, 27 Jan 2012 19:54:03 +0000 (19:54 +0000)
committerMark Brown <broonie@opensource.wolfsonmicro.com>
Fri, 17 Feb 2012 06:32:03 +0000 (22:32 -0800)
Only enable and disable the FLL when explicitly told to, supporting some
additional use cases and making the driver behaviour more standard.

Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
sound/soc/codecs/wm8962.c

index c36178b3d44192723b0f7126b88e1abbcbb0b8c3..3dba53ace6bd5b082a46f65c250345436d4e1504 100644 (file)
@@ -1832,65 +1832,6 @@ SOC_SINGLE_TLV("SPKOUTR Mixer DACR Volume", WM8962_SPEAKER_MIXER_5,
               4, 1, 0, inmix_tlv),
 };
 
-static int sysclk_event(struct snd_soc_dapm_widget *w,
-                       struct snd_kcontrol *kcontrol, int event)
-{
-       struct snd_soc_codec *codec = w->codec;
-       struct wm8962_priv *wm8962 = snd_soc_codec_get_drvdata(codec);
-       unsigned long timeout;
-       int src;
-       int fll;
-
-       /* Ignore attempts to run the event during startup */
-       if (codec->dapm.bias_level == SND_SOC_BIAS_OFF)
-               return 0;
-
-       src = snd_soc_read(codec, WM8962_CLOCKING2) & WM8962_SYSCLK_SRC_MASK;
-
-       switch (src) {
-       case 0:      /* MCLK */
-               fll = 0;
-               break;
-       case 0x200:  /* FLL */
-               fll = 1;
-               break;
-       default:
-               dev_err(codec->dev, "Unknown SYSCLK source %x\n", src);
-               return -EINVAL;
-       }
-
-       switch (event) {
-       case SND_SOC_DAPM_PRE_PMU:
-               if (fll) {
-                       try_wait_for_completion(&wm8962->fll_lock);
-
-                       snd_soc_update_bits(codec, WM8962_FLL_CONTROL_1,
-                                           WM8962_FLL_ENA, WM8962_FLL_ENA);
-
-                       timeout = msecs_to_jiffies(5);
-                       timeout = wait_for_completion_timeout(&wm8962->fll_lock,
-                                                             timeout);
-
-                       if (wm8962->irq && timeout == 0)
-                               dev_err(codec->dev,
-                                       "Timed out starting FLL\n");
-               }
-               break;
-
-       case SND_SOC_DAPM_POST_PMD:
-               if (fll)
-                       snd_soc_update_bits(codec, WM8962_FLL_CONTROL_1,
-                                           WM8962_FLL_ENA, 0);
-               break;
-
-       default:
-               BUG();
-               return -EINVAL;
-       }
-
-       return 0;
-}
-
 static int cp_event(struct snd_soc_dapm_widget *w,
                    struct snd_kcontrol *kcontrol, int event)
 {
@@ -2176,8 +2117,7 @@ SND_SOC_DAPM_INPUT("DMICDAT"),
 SND_SOC_DAPM_SUPPLY("MICBIAS", WM8962_PWR_MGMT_1, 1, 0, NULL, 0),
 
 SND_SOC_DAPM_SUPPLY("Class G", WM8962_CHARGE_PUMP_B, 0, 1, NULL, 0),
-SND_SOC_DAPM_SUPPLY("SYSCLK", WM8962_CLOCKING2, 5, 0, sysclk_event,
-                   SND_SOC_DAPM_PRE_PMU | SND_SOC_DAPM_POST_PMD),
+SND_SOC_DAPM_SUPPLY("SYSCLK", WM8962_CLOCKING2, 5, 0, NULL, 0),
 SND_SOC_DAPM_SUPPLY("Charge Pump", WM8962_CHARGE_PUMP_1, 0, 0, cp_event,
                    SND_SOC_DAPM_POST_PMU),
 SND_SOC_DAPM_SUPPLY("TOCLK", WM8962_ADDITIONAL_CONTROL_1, 0, 0, NULL, 0),
@@ -2888,8 +2828,7 @@ static int wm8962_set_fll(struct snd_soc_codec *codec, int fll_id, int source,
        struct _fll_div fll_div;
        unsigned long timeout;
        int ret;
-       int fll1 = snd_soc_read(codec, WM8962_FLL_CONTROL_1) & WM8962_FLL_ENA;
-       int sysclk = snd_soc_read(codec, WM8962_CLOCKING2) & WM8962_SYSCLK_ENA;
+       int fll1 = 0;
 
        /* Any change? */
        if (source == wm8962->fll_src && Fref == wm8962->fll_fref &&
@@ -2912,6 +2851,9 @@ static int wm8962_set_fll(struct snd_soc_codec *codec, int fll_id, int source,
        if (ret != 0)
                return ret;
 
+       /* Parameters good, disable so we can reprogram */
+       snd_soc_update_bits(codec, WM8962_FLL_CONTROL_1, WM8962_FLL_ENA, 0);
+
        switch (fll_id) {
        case WM8962_FLL_MCLK:
        case WM8962_FLL_BCLK:
@@ -2950,12 +2892,10 @@ static int wm8962_set_fll(struct snd_soc_codec *codec, int fll_id, int source,
 
        try_wait_for_completion(&wm8962->fll_lock);
 
-       if (sysclk)
-               fll1 |= WM8962_FLL_ENA;
 
        snd_soc_update_bits(codec, WM8962_FLL_CONTROL_1,
                            WM8962_FLL_FRAC | WM8962_FLL_REFCLK_SRC_MASK |
-                           WM8962_FLL_ENA, fll1);
+                           WM8962_FLL_ENA, fll1 | WM8962_FLL_ENA);
 
        dev_dbg(codec->dev, "FLL configured for %dHz->%dHz\n", Fref, Fout);