At Mon, 19 May 2008 12:37:10 +0200, I wrote:
At Mon, 19 May 2008 11:31:09 +0100, Tony Vroon wrote:
On Mon, 2008-05-19 at 12:23 +0200, Takashi Iwai wrote:
Any reason to remove this? Otherwise the internal speaker won't react together with the Master volume.
It seemed to only take the internal headphone output into account, not the dock headphone output. But if it's just to link the two volumes together I'll add it back. It will fix the compile warning as well. Its purpose was unclear to me.
Will the existing mechanism cope if 0x14 (internal HP output) is muted and 0x1B (dock HP output) is not?
The code there was to synchronize both HP and speaker outputs with the master switch. The master switch changes 0x14 mute status and calls automute. When no HP is plugged, the internal speaker is unmuted only when master switch is on -- that is, 0x14 is also unmuted.
The logic is a bit complex, and thus I suggested to replace with vmaster.
Maybe a cleaner fix is to remove the bind_sw and bind_vol hacks and use vmaster instead.
Is there example code I can look at please?
Add the individual mixer elements corresponding to HP, dock and PC-speaker switches instead of hackish master switch. Then a virtual master switch will be automatically created.
I'll create a test patch soon later...
On the second thought, it's also not so trivial in this case because of the combination with the jack detection code.
How about the patch below?
Note that the mute bits of 0x14 and 0x1b follow only to the state of master switch. It's on/off regardless of jack detection. Only the internal speaker is affected via jack-detection.
Takashi
---
diff -r e2b60934d1c8 pci/hda/patch_realtek.c --- a/pci/hda/patch_realtek.c Mon May 19 12:32:25 2008 +0200 +++ b/pci/hda/patch_realtek.c Mon May 19 12:54:26 2008 +0200 @@ -8757,35 +8757,39 @@ }, };
-/* mute/unmute internal speaker according to the hp jack and mute state */ +/* mute/unmute internal speaker according to the hp jacks and mute state */ static void alc262_fujitsu_automute(struct hda_codec *codec, int force) { struct alc_spec *spec = codec->spec; unsigned int mute;
if (force || !spec->sense_updated) { - unsigned int present_int_hp, present_dock_hp; + unsigned int present; /* need to execute and sync at first */ snd_hda_codec_read(codec, 0x14, 0, AC_VERB_SET_PIN_SENSE, 0); - present_int_hp = snd_hda_codec_read(codec, 0x14, 0, - AC_VERB_GET_PIN_SENSE, 0); - snd_hda_codec_read(codec, 0x1B, 0, AC_VERB_SET_PIN_SENSE, 0); - present_dock_hp = snd_hda_codec_read(codec, 0x1b, 0, - AC_VERB_GET_PIN_SENSE, 0); - spec->jack_present = (present_int_hp & 0x80000000) != 0; - spec->jack_present |= (present_dock_hp & 0x80000000) != 0; + /* check laptop HP jack */ + present = snd_hda_codec_read(codec, 0x14, 0, + AC_VERB_GET_PIN_SENSE, 0); + /* need to execute and sync at first */ + snd_hda_codec_read(codec, 0x1b, 0, AC_VERB_SET_PIN_SENSE, 0); + /* check docking HP jack */ + present |= snd_hda_codec_read(codec, 0x1b, 0, + AC_VERB_GET_PIN_SENSE, 0); + if (present & AC_PINSENSE_PRESENCE) + spec->jack_present = 1; + else + spec->jack_present = 0; spec->sense_updated = 1; } - if (spec->jack_present) { - /* mute internal speaker */ - snd_hda_codec_amp_stereo(codec, 0x15, HDA_OUTPUT, 0, - HDA_AMP_MUTE, HDA_AMP_MUTE); - } else { - /* unmute internal speaker if necessary */ + /* unmute internal speaker only if both HPs are unplugged and + * master switch is on + */ + if (spec->jack_present) + mute = HDA_AMP_MUTE; + else mute = snd_hda_codec_amp_read(codec, 0x14, 0, HDA_OUTPUT, 0); - snd_hda_codec_amp_stereo(codec, 0x15, HDA_OUTPUT, 0, - HDA_AMP_MUTE, mute); - } + snd_hda_codec_amp_stereo(codec, 0x15, HDA_OUTPUT, 0, + HDA_AMP_MUTE, mute); }
/* unsolicited event for HP jack sensing */ @@ -8794,6 +8798,11 @@ { if ((res >> 26) != ALC_HP_EVENT) return; + alc262_fujitsu_automute(codec, 1); +} + +static void alc262_fujitsu_init_hook(struct hda_codec *codec) +{ alc262_fujitsu_automute(codec, 1); }
@@ -9570,6 +9579,7 @@ .channel_mode = alc262_modes, .input_mux = &alc262_fujitsu_capture_source, .unsol_event = alc262_fujitsu_unsol_event, + .init_hook = alc262_fujitsu_init_hook, }, [ALC262_HP_BPC] = { .mixers = { alc262_HP_BPC_mixer },