[alsa-devel] Clevo P170HM / Sager NP8170 audio
Raymond Yau
superquad.vortex2 at gmail.com
Mon Nov 28 01:42:12 CET 2011
2011/11/25 Julian Sikorski <belegdol at gmail.com>:
> W dniu 25.11.2011 07:39, Raymond Yau pisze:
>> 2011/11/22 Julian Sikorski <belegdol at gmail.com>:
>>
>> Don't have any realtek codec, not sure why hda-jack-sense-test.py
>> locked up when you use user_pin_configs
>>
>>>>> I then applied your patch on top of Fedora 3.1.1 kernel (crude patch
>>>>> attached). After reboot:
>>>>> * hda-jack-sense-test.py was still working, without lockups this time
>>>>> * alsamixer -c0 revealed a new "Front" slider, which was controlling the
>>>>> volume on 0x17
>>>>> * there was still no 8-channel mode (probably related to what you wrote
>>>>> below)
>>>>> * there were more options for auto mute (Disabled, Speaker Only, Line
>>>>> Out+Speaker)
>>>>> * here is the new alsa-info.sh:
>>>>> http://www.alsa-project.org/db/?f=923b75ad3997dc8f5878852e327f9b999a196052
>>>>>
>>>>
>>>>
>>>>>>
>>>>>> To add the missing pin default of side jack 0x17
>>>>>>
>>>>>> ALC662_FIXUP_ASUS_MODE8,
>>>>>> + ALC892_FIXUP_CLEVO_4ST_8CH,
>>>>>> };
>>>>>>
>>>>>> static const struct alc_fixup alc662_fixups[] = {
>>>>>>
>>>>>>
>>>>>> + [ALC892_FIXUP_CLEVO_4ST_8CH] = {
>>>>>> + .type = ALC_FIXUP_PINS,
>>>>>> + .v.pins = (const struct alc_pincfg[]) {
>>>>>> +/*
>>>>>> + need to be same location as the other jack
>>>>>> + may need to change the default association and sequence since
>>>>>> + Lower Default Association values would be higher in priority for resources
>>>>>> + such as processing nodes or Input and Output Converters.
>>>>>> + A value of 0000b is reserved and should not be used
>>>>>> +*/
>>>>>> + { 0x17, 0x01011013 }, /* Side */
>>>>>> + { }
>>>>>> + },
>>>>>> + },
>>>>>>
>>>>>>
>>>>>> + SND_PCI_QUIRK(0x1558, 0x5102, "Clevo P150HM", ALC892_FIXUP_CLEVO_4ST_8CH),
>>>>>> + SND_PCI_QUIRK(0x1558, 0x7100, "Clevo P170HM", ALC892_FIXUP_CLEVO_4ST_8CH),
>>>>>>
>>>>>> SImilar case are those notebooks hda-emu/codecs/alc1200-msi-gx620
>>>>>> which has 1 hp(green), 1 line-in(blue), 1 ext-mic(pink) and 1 line
>>>>>> out(grey) at same location (ext rear) for surround71 (and also 1
>>>>>> speakers, 1 int mic)
>>>>>>
>>>>>>
>>>>>> hda_codec: ALC1200: BIOS auto-probing.
>>>>>> hda_codec: ALC1200: SKU not ready 0x598301f0
>>>>>> autoconfig: line_outs=1 (0x17/0x0/0x0/0x0/0x0) type:line
>>>>>> speaker_outs=1 (0x1b/0x0/0x0/0x0/0x0)
>>>>>> hp_outs=1 (0x14/0x0/0x0/0x0/0x0)
>>>>>> mono: mono_out=0x0
>>>>>> dig-out=0x1e/0x0
>>>>>> inputs: Mic=0x18 Internal Mic=0x19 Line=0x1a
>>>>>>
>>>>>>
>>>>>> CTRL: add: Line-Out Jack:0
>>>>>> CTRL: add: Headphone Jack:0
>>>>>> CTRL: add: Mic Jack:0
>>>>>> CTRL: add: Line Jack:0
>>>>>>
>>>>>>> get 1
>>>>>> 1 Channel Mode:0
>>>>>> ITEM: 0:2ch, 1:4ch, 2:6ch, VAL: [2ch]
>>>>>>
>>
>> After the pin fixup, your notebook is almost like alc1200 msi gx460
>> except speaker and hp are swapped, this mean that those volume
>> controls and switches are similar to targa_mixer , targa_8ch_mixer of
>> model="targa-8ch-dig"
>>
>>>>>>
>>>>>> This mean that when switch to 6/8 channel mode, it will conflict with
>>>>>> automic detection since the pin cap alc892 's mic jack does not
>>>>>> support impedance sense and the driver does not know the plug is mic
>>>>>> or speaker by measuring the impedance
>>>>>>
>>>>>> The driver have to disable automic detection when user switch "channel
>>>>>> mode" to 6ch/8ch as the user should still able to use the internal mic
>>>>>> when the external mic jack is retasked as output
>>>>>>
>>
>> The are some difference between p170hm and your p150hm if p170hm has
>> 5.1 speakers which still can have auto mic detection enabled.
>>
>>
>> you can add the following code in alc_auto_ch_mode_put() for debugging
>> and need to switch "auto mute mode" to "speaker" for 8 channels
>>
>>
>> if (spec->need_dac_fix && !spec->const_channel_count)
>> spec->multiout.num_dacs = spec->multiout.max_channels / 2;
>>
>> + printk(KERN_INFO "ch = %d\n",ch);
>> + printk(KERN_INFO "max channel %d\n",spec->multiout.max_channels);
>> + for (i=0; i<spec->autocfg.hp_outs; i++)
>> + printk(KERN_INFO "hp dac(%d) %x pin %x\n", i,
>> spec->multiout.hp_out_nid[i], spec->autocfg.hp_pins[i]);
>> + for (i=0; i<spec->multiout.num_dacs; i++)
>> + printk(KERN_INFO "private_dac(%d) %x\n", i, spec->private_dac_nids[i]);
>> + for (i=0; i<spec->autocfg.line_outs; i++)
>> + printk(KERN_INFO "line_out(%d) pin %x\n", i, spec->autocfg.line_out_pins[i]);
>> + for (i=0; i<spec->multiout.num_dacs-1; i++)
>> + printk(KERN_INFO "multi_io(%d) dac %x pin %x\n", i,
>> spec->multi_io[i].dac, spec->multi_io[i].pin);
>> + for (i=0; i<spec->autocfg.speaker_outs; i++)
>> + printk(KERN_INFO "extra_out(%d) dac %x pin %x\n", i,
>> spec->multiout.extra_out_nid[i], spec->autocfg.speaker_pins[i]);
>> return 1
>
> This is what appears in /var/log/messages if you go from 8 to 2 channels
> and back:
>
> Nov 25 13:05:11 snowball2 kernel: [ 1515.853640] ch = 2
> Nov 25 13:05:11 snowball2 kernel: [ 1515.853642] max channel 6
> Nov 25 13:05:11 snowball2 kernel: [ 1515.853643] hp dac(0) 0 pin 1b
> Nov 25 13:05:11 snowball2 kernel: [ 1515.853644] private_dac(0) 2
> Nov 25 13:05:11 snowball2 kernel: [ 1515.853645] private_dac(1) 3
> Nov 25 13:05:11 snowball2 kernel: [ 1515.853646] private_dac(2) 4
> Nov 25 13:05:11 snowball2 kernel: [ 1515.853647] private_dac(3) 5
> Nov 25 13:05:11 snowball2 kernel: [ 1515.853648] line_out(0) pin 17
> Nov 25 13:05:11 snowball2 kernel: [ 1515.853649] multi_io(0) dac 3 pin 1a
> Nov 25 13:05:11 snowball2 kernel: [ 1515.853650] multi_io(1) dac 4 pin 18
> Nov 25 13:05:11 snowball2 kernel: [ 1515.853651] multi_io(2) dac 5 pin 17
> Nov 25 13:05:11 snowball2 kernel: [ 1515.853652] extra_out(0) dac 0 pin 14
> Nov 25 13:05:24 snowball2 kernel: [ 1528.952829] ch = 1
> Nov 25 13:05:24 snowball2 kernel: [ 1528.952835] max channel 4
> Nov 25 13:05:24 snowball2 kernel: [ 1528.952840] hp dac(0) 0 pin 1b
> Nov 25 13:05:24 snowball2 kernel: [ 1528.952843] private_dac(0) 2
> Nov 25 13:05:24 snowball2 kernel: [ 1528.952846] private_dac(1) 3
> Nov 25 13:05:24 snowball2 kernel: [ 1528.952849] private_dac(2) 4
> Nov 25 13:05:24 snowball2 kernel: [ 1528.952852] private_dac(3) 5
> Nov 25 13:05:24 snowball2 kernel: [ 1528.952855] line_out(0) pin 17
> Nov 25 13:05:24 snowball2 kernel: [ 1528.952859] multi_io(0) dac 3 pin 1a
> Nov 25 13:05:24 snowball2 kernel: [ 1528.952863] multi_io(1) dac 4 pin 18
> Nov 25 13:05:24 snowball2 kernel: [ 1528.952866] multi_io(2) dac 5 pin 17
> Nov 25 13:05:24 snowball2 kernel: [ 1528.952870] extra_out(0) dac 0 pin 14
> Nov 25 13:05:26 snowball2 kernel: [ 1531.378952] ch = 0
> Nov 25 13:05:26 snowball2 kernel: [ 1531.378958] max channel 2
> Nov 25 13:05:26 snowball2 kernel: [ 1531.378963] hp dac(0) 0 pin 1b
> Nov 25 13:05:26 snowball2 kernel: [ 1531.378966] private_dac(0) 2
> Nov 25 13:05:26 snowball2 kernel: [ 1531.378969] private_dac(1) 3
> Nov 25 13:05:26 snowball2 kernel: [ 1531.378972] private_dac(2) 4
> Nov 25 13:05:26 snowball2 kernel: [ 1531.378975] private_dac(3) 5
> Nov 25 13:05:26 snowball2 kernel: [ 1531.378978] line_out(0) pin 17
> Nov 25 13:05:26 snowball2 kernel: [ 1531.378982] multi_io(0) dac 3 pin 1a
> Nov 25 13:05:26 snowball2 kernel: [ 1531.379002] multi_io(1) dac 4 pin 18
> Nov 25 13:05:26 snowball2 kernel: [ 1531.379006] multi_io(2) dac 5 pin 17
> Nov 25 13:05:26 snowball2 kernel: [ 1531.379010] extra_out(0) dac 0 pin 14
> Nov 25 13:06:29 snowball2 kernel: [ 1593.746269] ch = 1
> Nov 25 13:06:29 snowball2 kernel: [ 1593.746275] max channel 4
> Nov 25 13:06:29 snowball2 kernel: [ 1593.746279] hp dac(0) 0 pin 1b
> Nov 25 13:06:29 snowball2 kernel: [ 1593.746283] private_dac(0) 2
> Nov 25 13:06:29 snowball2 kernel: [ 1593.746286] private_dac(1) 3
> Nov 25 13:06:29 snowball2 kernel: [ 1593.746289] private_dac(2) 4
> Nov 25 13:06:29 snowball2 kernel: [ 1593.746291] private_dac(3) 5
> Nov 25 13:06:29 snowball2 kernel: [ 1593.746295] line_out(0) pin 17
> Nov 25 13:06:29 snowball2 kernel: [ 1593.746298] multi_io(0) dac 3 pin 1a
> Nov 25 13:06:29 snowball2 kernel: [ 1593.746302] multi_io(1) dac 4 pin 18
> Nov 25 13:06:29 snowball2 kernel: [ 1593.746306] multi_io(2) dac 5 pin 17
> Nov 25 13:06:29 snowball2 kernel: [ 1593.746310] extra_out(0) dac 0 pin 14
> Nov 25 13:06:31 snowball2 kernel: [ 1596.174127] ch = 2
> Nov 25 13:06:31 snowball2 kernel: [ 1596.174133] max channel 6
> Nov 25 13:06:31 snowball2 kernel: [ 1596.174195] hp dac(0) 0 pin 1b
> Nov 25 13:06:31 snowball2 kernel: [ 1596.174199] private_dac(0) 2
> Nov 25 13:06:31 snowball2 kernel: [ 1596.174202] private_dac(1) 3
> Nov 25 13:06:31 snowball2 kernel: [ 1596.174205] private_dac(2) 4
> Nov 25 13:06:31 snowball2 kernel: [ 1596.174208] private_dac(3) 5
> Nov 25 13:06:31 snowball2 kernel: [ 1596.174211] line_out(0) pin 17
> Nov 25 13:06:31 snowball2 kernel: [ 1596.174215] multi_io(0) dac 3 pin 1a
> Nov 25 13:06:31 snowball2 kernel: [ 1596.174219] multi_io(1) dac 4 pin 18
> Nov 25 13:06:31 snowball2 kernel: [ 1596.174223] multi_io(2) dac 5 pin 17
> Nov 25 13:06:31 snowball2 kernel: [ 1596.174226] extra_out(0) dac 0 pin 14
> Nov 25 13:06:33 snowball2 kernel: [ 1598.111893] ch = 3
> Nov 25 13:06:33 snowball2 kernel: [ 1598.111900] max channel 8
> Nov 25 13:06:33 snowball2 kernel: [ 1598.111904] hp dac(0) 0 pin 1b
> Nov 25 13:06:33 snowball2 kernel: [ 1598.111908] private_dac(0) 2
> Nov 25 13:06:33 snowball2 kernel: [ 1598.111911] private_dac(1) 3
> Nov 25 13:06:33 snowball2 kernel: [ 1598.111914] private_dac(2) 4
> Nov 25 13:06:33 snowball2 kernel: [ 1598.111917] private_dac(3) 5
> Nov 25 13:06:33 snowball2 kernel: [ 1598.111920] line_out(0) pin 17
> Nov 25 13:06:33 snowball2 kernel: [ 1598.111924] multi_io(0) dac 3 pin 1a
> Nov 25 13:06:33 snowball2 kernel: [ 1598.111927] multi_io(1) dac 4 pin 18
> Nov 25 13:06:33 snowball2 kernel: [ 1598.111931] multi_io(2) dac 5 pin 17
> Nov 25 13:06:33 snowball2 kernel: [ 1598.111935] extra_out(0) dac 0 pin 14
>
>>
>> Try the following code which try to assign dac 02, 03, 04, and 05 to
>> spec->private_dac front, surround, clfe and side.
>>
>> static int alc_auto_fill_dac_nids(struct hda_codec *codec)
>> {
>> ...
>> again:
>> /* set num_dacs once to full for alc_auto_look_for_dac() */
>> spec->multiout.num_dacs = cfg->line_outs;
>> spec->multiout.hp_out_nid[0] = 0;
>> spec->multiout.extra_out_nid[0] = 0;
>> memset(spec->private_dac_nids, 0, sizeof(spec->private_dac_nids));
>> spec->multiout.dac_nids = spec->private_dac_nids;
>>
>> + if (cfg->hp_outs == 1 && cfg->line_outs == 1 &&
>> + cfg->num_inputs >= 3 && cfg->speaker_outs >= 1) {
>> +/* assign 0x02 as Front DAC by using speaker or hp pins*/
>> + spec->private_dac_nids[0] = alc_auto_look_for_dac(codec,
>> cfg->speaker_pins[0]);
>> +/* hp , mic and line-in at the same location
>> + use alc_auto_fill_multi_ios() to assign surround and clfe dacs
>> +*/
>> + defcfg = snd_hda_codec_get_pincfg(codec, cfg->hp_pins[0]);
>> + location = get_defcfg_location(defcfg);
>> + spec->multiout.num_dacs = 1;
>> + num_pins = alc_auto_fill_multi_ios(codec, location, 1);
>> + if (num_pins > 0) {
>> + spec->multi_ios = num_pins;
>> + spec->ext_channel_count = 2;
>> + spec->multiout.num_dacs = num_pins + 1;
>> + }
>> +/* an ugly hack to add side as multi io even side does not support input
>> + set pin ctl to PIN_OUT to set multi_io[2].ctl_in to PIN_OUT */
>> + if (location == get_defcfg_location(snd_hda_codec_get_pincfg(codec,
>> cfg->line_out_pins[0]))) {
>> + spec->private_dac_nids[3] = alc_auto_look_for_dac(codec,
>> cfg->line_out_pins[0]);
>> + spec->multi_io[2].dac = spec->private_dac_nids[3];
>> + spec->multi_io[2].pin = cfg->line_out_pins[0];
>> + snd_hda_codec_write(codec, cfg->line_out_pins[0], 0,
>> AC_VERB_SET_PIN_WIDGET_CONTROL, PIN_OUT);
>> + spec->multi_ios++;
>> + spec->multiout.num_dacs++;
>> + }
>> + return 0;
>> + }
>
> It is much better now. I applied the attached patch and 2, 4, 6 and 8
> channel sound is working, enabling outputs incrementally as I change the
> number of channels in alsamixer. Observations:
> * there is no mute for side channel
> * no matter if Auto-Mute is set to Speaker or Line Out+Speaker, speakers
> will mute upon plugging a jack into either headphone or side/spdif socket
> * there is a "front" slider which controls internal speakers and HP
> jack, but the mute control under it does nothing. There are separate
> mute controls for HP and Speakers, which work as advertised.
> This is the alsa-info.sh output:
> http://www.alsa-project.org/db/?f=7df8ccac0a5ae751110f7b7e56f9e0ba5cc3b302
> Looks like we are almost there, thanks for all the input so far!
There is a bug in hda-emu which cannot detect two playback switches
"Front" and "Side" create at same switch pin
Try add the following to dump the value of switch pin
In function alc_auto_create_multi_out_ctls()
+ printk(KERN_INFO "%s playback volume %x pin %x\n",name,vol,pin);
err = alc_auto_add_stereo_vol(codec, name, index, vol);
if (err < 0)
return err;
+ printk(KERN_INFO "%s playback switch %x\n",name,sw);
err = alc_auto_add_stereo_sw(codec, name, index, sw);
if (err < 0)
return err;
>>>>>>>> Plug the front speaker cables into the Headphone-Out Jack.
>>>>>>>>
>>>>>>>> • Line-In Jack = Rear Speaker Out
>>>>>>>> • Microphone-In Jack = Center/Subwoofer Speaker Out
>>>>>>>> • S/PDIF-Out Jack = Side Speaker Out (for 7.1 Surround Sound Only)
Method 1) HP Playback volume for HP jack and Speaker Playback Volume
fix the "Side Playback Switch" and "Front playback Switch"
In function alc_auto_create_multi_out_ctls()
else
- pin = cfg->line_out_pins[i];
+ if (cfg->speaker_pins[0] &&
+ get_defcfg_location(snd_hda_codec_get_pincfg(codec, cfg->hp_pins[0])) ==
get_defcfg_location(snd_hda_codec_get_pincfg(codec, cfg->line_out_pins[0])))
+ pin = cfg->speaker_pins[0];
+ else
+ pin = cfg->line_out_pins[i];
add "Headpone Playback Volume" by using dac 0x25
In function alc_auto_fill_dac_nids() and the end of previous patch in
previous email
spec->multi_ios++;
spec->multiout.num_dacs++;
}
+ spec->multiout.hp_out_nid[0] = alc_auto_look_for_dac(codec, cfg->hp_pins[0]);
return 0;
>>>>>>>> • S/PDIF-Out Jack = Side Speaker Out (for 7.1 Surround Sound Only)
assign dac spec->alt_dac_nid for creating alt playback device
aplay -Dhw:0,2 any.wav
Try either a) or b) but not both
a) use "side jack" for "alt playback" but device 2 be disabled when
channel mode is 8ch
In function alc_auto_fill_dac_nids()
+ spec->alt_dac_nid = spec->private_dac_nids[3];
return 0;
in function alc_set_multi_io()
} else {
+ if ( i < 2 ) {
- if (get_wcaps(codec, nid) & AC_WCAP_OUT_AMP)
snd_hda_codec_amp_stereo(codec, nid, HDA_OUTPUT, 0,
HDA_AMP_MUTE, HDA_AMP_MUTE);
- snd_hda_codec_update_cache(codec, nid, 0,
AC_VERB_SET_PIN_WIDGET_CONTROL,
spec->multi_io[idx].ctl_in);
+ if (get_wcaps(codec, nid) & AC_WCAP_OUT_AMP)
snd_hda_codec_amp_stereo(codec, nid, HDA_OUTPUT, 0,
HDA_AMP_MUTE, HDA_AMP_MUTE);
+ snd_hda_codec_update_cache(codec, nid, 0,
AC_VERB_SET_PIN_WIDGET_CONTROL,
spec->multi_io[idx].ctl_in);
+ }
b) use "Headphone" for "alt playback"
In function alc_auto_fill_dac_nids()
+ spec->alt_dac_nid = spec->multiout.hp_out_nid[0];
return 0;
More information about the Alsa-devel
mailing list