ASoC: pxa: magician: Update locking around use of DAPM pin API
The pin updates in this driver look like they are intended to be done atomically, update to do so. Signed-off-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com> Signed-off-by: Mark Brown <broonie@linaro.org>
This commit is contained in:

committed by
Mark Brown

parent
4d9e73488d
commit
f71b0fbe52
@@ -45,27 +45,31 @@ static void magician_ext_control(struct snd_soc_codec *codec)
|
|||||||
{
|
{
|
||||||
struct snd_soc_dapm_context *dapm = &codec->dapm;
|
struct snd_soc_dapm_context *dapm = &codec->dapm;
|
||||||
|
|
||||||
|
snd_soc_dapm_mutex_lock(dapm);
|
||||||
|
|
||||||
if (magician_spk_switch)
|
if (magician_spk_switch)
|
||||||
snd_soc_dapm_enable_pin(dapm, "Speaker");
|
snd_soc_dapm_enable_pin_unlocked(dapm, "Speaker");
|
||||||
else
|
else
|
||||||
snd_soc_dapm_disable_pin(dapm, "Speaker");
|
snd_soc_dapm_disable_pin_unlocked(dapm, "Speaker");
|
||||||
if (magician_hp_switch)
|
if (magician_hp_switch)
|
||||||
snd_soc_dapm_enable_pin(dapm, "Headphone Jack");
|
snd_soc_dapm_enable_pin_unlocked(dapm, "Headphone Jack");
|
||||||
else
|
else
|
||||||
snd_soc_dapm_disable_pin(dapm, "Headphone Jack");
|
snd_soc_dapm_disable_pin_unlocked(dapm, "Headphone Jack");
|
||||||
|
|
||||||
switch (magician_in_sel) {
|
switch (magician_in_sel) {
|
||||||
case MAGICIAN_MIC:
|
case MAGICIAN_MIC:
|
||||||
snd_soc_dapm_disable_pin(dapm, "Headset Mic");
|
snd_soc_dapm_disable_pin_unlocked(dapm, "Headset Mic");
|
||||||
snd_soc_dapm_enable_pin(dapm, "Call Mic");
|
snd_soc_dapm_enable_pin_unlocked(dapm, "Call Mic");
|
||||||
break;
|
break;
|
||||||
case MAGICIAN_MIC_EXT:
|
case MAGICIAN_MIC_EXT:
|
||||||
snd_soc_dapm_disable_pin(dapm, "Call Mic");
|
snd_soc_dapm_disable_pin_unlocked(dapm, "Call Mic");
|
||||||
snd_soc_dapm_enable_pin(dapm, "Headset Mic");
|
snd_soc_dapm_enable_pin_unlocked(dapm, "Headset Mic");
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
|
|
||||||
snd_soc_dapm_sync(dapm);
|
snd_soc_dapm_sync_unlocked(dapm);
|
||||||
|
|
||||||
|
snd_soc_dapm_mutex_unlock(dapm);
|
||||||
}
|
}
|
||||||
|
|
||||||
static int magician_startup(struct snd_pcm_substream *substream)
|
static int magician_startup(struct snd_pcm_substream *substream)
|
||||||
|
Reference in New Issue
Block a user