At Tue, 3 Dec 2013 15:52:37 +0800, Kailang wrote:

Hi David,

The verb was ADC digital mute(bit 6 default 1). Switch internal mic and headset mic will run alc_headset_mode_default. The coef index 0x11 will set to 0x0041. Because headset mode was fixed type. It doesn't need to run alc_determine_headset_type. So, the value still keep 0x0041. ADC was muted. Your patch will solve this errors.

OK, I added your comment in the changelog and add Cc to stable, then applied to for-linus branch now.

thanks,

Takashi

Thanks. Kailang

...

By trial and error, I found this patch could work around an issue where the headset mic would stop working if you switch between the internal mic and the headset mic, and the internal mic was muted.

It still takes a second or two before the headset mic actually starts working, but still better than nothing.

I'd like to add stable to this patch, but I'd like Kailang's ack first because I don't know what the verbs really do.

BugLink: https://bugs.launchpad.net/bugs/1256840 Signed-off-by: David Henningsson david.henningsson@canonical.com

---

sound/pci/hda/patch_realtek.c | 2 ++ 1 file changed, 2 insertions(+)

diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 537991c..85d1c8b 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -3338,6 +3338,7 @@ static void alc_headset_mode_ctia(struct hda_codec *codec) alc_write_coef_idx(codec, 0x18, 0x7388); break; case 0x10ec0668:

• alc_write_coef_idx(codec, 0x11, 0x0001);

alc_write_coef_idx(codec, 0x15, 0x0d60); alc_write_coef_idx(codec, 0xc3, 0x0000); break; @@ -3366,6 +3367,7 @@ static void alc_headset_mode_omtp(struct hda_codec *codec) alc_write_coef_idx(codec, 0x18, 0x7388); break; case 0x10ec0668:

• alc_write_coef_idx(codec, 0x11, 0x0001);

alc_write_coef_idx(codec, 0x15, 0x0d50); alc_write_coef_idx(codec, 0xc3, 0x0000); break; -- 1.7.9.5

------Please consider the environment before printing this e-mail.