-----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);
  

  regmap_update_bits(rt5640->regmap, RT5640_PR_BASE + 0x1c, 0xf000, 0xf000); break;

@@ -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_DAPM_PGA("HP L Amp", RT5640_PWR_ANLG1, RT5640_PWR_HP_L_BIT, 0, NULL, 0), SND_SOC_DAPM_PGA("HP R Amp", RT5640_PWR_ANLG1, RT5640_PWR_HP_R_BIT, 0, NULL, 0), SND_SOC_DAPM_SUPPLY("Improve SPK Amp Drv", RT5640_PWR_DIG1,

• SND_SOC_NOPM, 0, spk_event,
  

• RT5640_PWR_CLS_D_BIT, 0, spk_event,
  

  SND_SOC_DAPM_PRE_PMD | SND_SOC_DAPM_POST_PMU), /* Output Lines */ SND_SOC_DAPM_OUTPUT("SPOLP"),

---

Alsa-devel mailing list Alsa-devel@alsa-project.org http://mailman.alsa-project.org/mailman/listinfo/alsa-devel