-----Original Message----- From: alsa-devel-bounces@alsa-project.org [mailto:alsa-devel-bounces@alsa-project.org] On Behalf Of Stephen Warren Sent: Saturday, July 27, 2013 12:58 AM To: Lars-Peter Clausen Cc: Bard Liao; Linux-ALSA Subject: Re: [alsa-devel] Broken headphone and speaker amplifier suppliesfor the rt5640 driver
On 07/26/2013 09:06 AM, Lars-Peter Clausen wrote:
Hi,
The rt5640 driver sets for both the headphone and speaker amplifier widget the shift to SND_SOC_NOPM, which is -1. This doesn't make much sense and breaks compilation with some changes I've made.
I guess for the heaphone amplifier the right fix is to set the shift to RT5640_PWR_HA_BIT ...
If I make those changes (see below for the exact patch I tried), then both headphones and speakers do appear to still work fine. Bard will have to comment on whether the changes are actually correct though:-)
The control bits for headphone amplifier and speaker amplifier are both correct. But rt5640 need a sequence for headphone depop. So, I would like to do something like this. - SND_SOC_DAPM_SUPPLY("Improve HP Amp Drv", RT5640_PWR_ANLG1, - SND_SOC_NOPM, 0, NULL, 0), + SND_SOC_DAPM_SUPPLY_S("Improve HP Amp Drv", 1, SND_SOC_NOPM, + 0, 0, rt5640_hp_power_event, + SND_SOC_DAPM_PRE_PMD | SND_SOC_DAPM_POST_PMU),
I will send a patch ASAP.
Thanks.
diff --git a/sound/soc/codecs/rt5640.c b/sound/soc/codecs/rt5640.c index ce585e3..e3ab0b2 100644 --- a/sound/soc/codecs/rt5640.c +++ b/sound/soc/codecs/rt5640.c @@ -899,8 +899,6 @@ static int spk_event(struct snd_soc_dapm_widget *w,
switch (event) { case SND_SOC_DAPM_POST_PMU:
regmap_update_bits(rt5640->regmap, RT5640_PWR_DIG1,0x0001, 0x0001);@@ -908,8 +906,6 @@ static int spk_event(struct snd_soc_dapm_widget
*w,
case SND_SOC_DAPM_PRE_PMD: regmap_update_bits(rt5640->regmap, RT5640_PR_BASE + 0x1c, 0xf000, 0x0000);
regmap_update_bits(rt5640->regmap, RT5640_PWR_DIG1,0x0001, 0x0000);break;
default:
@@ -1159,13 +1155,13 @@ static const struct snd_soc_dapm_widget
rt5640_dapm_widgets[] = {
SND_SOC_DAPM_SUPPLY("Improve MONO Amp Drv",
RT5640_PWR_ANLG1,
RT5640_PWR_MA_BIT, 0, NULL, 0),SND_SOC_DAPM_SUPPLY("Improve HP Amp Drv", RT5640_PWR_ANLG1,
SND_SOC_NOPM, 0, NULL, 0),
RT5640_PWR_HA_BIT, 0, NULL, 0),
SND_SOC_NOPM, 0, spk_event,
RT5640_PWR_CLS_D_BIT, 0, spk_event,
Alsa-devel mailing list Alsa-devel@alsa-project.org http://mailman.alsa-project.org/mailman/listinfo/alsa-devel