ASoC: wm8994: Don't lock CODEC mutex to do DAPM sync
authorMark Brown <broonie@opensource.wolfsonmicro.com>
Mon, 12 Mar 2012 16:34:35 +0000 (16:34 +0000)
committerMark Brown <broonie@opensource.wolfsonmicro.com>
Sun, 1 Apr 2012 10:28:23 +0000 (11:28 +0100)
DAPM now has a DAPM-level lock which it manages itself so we don't need
to take the CODEC mutex to call DAPM any more.  Also remove a redundant
call to snd_soc_dapm_sync(), jack reporting also triggers a DAPM sync.

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

index 7c49642af05249197f233c94832a16e6792200f5..fbcaf499b0ff76bddd99396943273a6f1d221bc9 100644 (file)
@@ -3247,9 +3247,6 @@ static void wm8958_default_micdet(u16 status, void *data)
 
                wm8958_micd_set_rate(codec);
 
-               snd_soc_jack_report(wm8994->micdet[0].jack, SND_JACK_HEADPHONE,
-                                   SND_JACK_HEADSET);
-
                /* If we have jackdet that will detect removal */
                if (wm8994->jackdet) {
                        mutex_lock(&wm8994->accdet_lock);
@@ -3262,14 +3259,13 @@ static void wm8958_default_micdet(u16 status, void *data)
 
                        mutex_unlock(&wm8994->accdet_lock);
 
-                       if (wm8994->pdata->jd_ext_cap) {
-                               mutex_lock(&codec->mutex);
+                       if (wm8994->pdata->jd_ext_cap)
                                snd_soc_dapm_disable_pin(&codec->dapm,
                                                         "MICBIAS2");
-                               snd_soc_dapm_sync(&codec->dapm);
-                               mutex_unlock(&codec->mutex);
-                       }
                }
+
+               snd_soc_jack_report(wm8994->micdet[0].jack, SND_JACK_HEADPHONE,
+                                   SND_JACK_HEADSET);
        }
 
        /* Report short circuit as a button */
@@ -3358,16 +3354,11 @@ static irqreturn_t wm1811_jackdet_irq(int irq, void *data)
 
        /* If required for an external cap force MICBIAS on */
        if (wm8994->pdata->jd_ext_cap) {
-               mutex_lock(&codec->mutex);
-
                if (present)
                        snd_soc_dapm_force_enable_pin(&codec->dapm,
                                                      "MICBIAS2");
                else
                        snd_soc_dapm_disable_pin(&codec->dapm, "MICBIAS2");
-
-               snd_soc_dapm_sync(&codec->dapm);
-               mutex_unlock(&codec->mutex);
        }
 
        if (present)