From: Julian Sikorski <belegdol@gmail.com>
To: alsa-devel@alsa-project.org
Subject: Re: Clevo P170HM / Sager NP8170 audio
Date: Mon, 28 Nov 2011 11:27:44 +0100 [thread overview]
Message-ID: <javnn0$m3$1@dough.gmane.org> (raw)
In-Reply-To: <CAN8cciYn_CSgrXHThe=TRBmC98e_XhkT=W8f=UaPp5Tg-c7vSw@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 17149 bytes --]
W dniu 28.11.2011 01:42, Raymond Yau pisze:
> 2011/11/25 Julian Sikorski <belegdol@gmail.com>:
>> W dniu 25.11.2011 07:39, Raymond Yau pisze:
>>> 2011/11/22 Julian Sikorski <belegdol@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 ) {
Patched kernel does not build (the patch I used is attached):
sound/pci/hda/patch_realtek.c: In function 'alc_set_multi_io':
sound/pci/hda/patch_realtek.c:3660:8: error: 'i' undeclared (first use
in this function)
sound/pci/hda/patch_realtek.c:3660:8: note: each undeclared identifier
is reported only once for each function it appears in
make[3]: *** [sound/pci/hda/patch_realtek.o] Error 1
Regards,
Julian
> - 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;
[-- Attachment #2: clevo-0x17-rawhide.patch --]
[-- Type: text/x-patch, Size: 6194 bytes --]
diff -up ./sound/pci/hda/patch_realtek.c.clevo ./sound/pci/hda/patch_realtek.c
--- ./sound/pci/hda/patch_realtek.c.clevo 2011-11-28 10:38:29.643369001 +0100
+++ ./sound/pci/hda/patch_realtek.c 2011-11-28 10:52:38.741831448 +0100
@@ -3053,6 +3053,38 @@ static int alc_auto_fill_dac_nids(struct
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++;
+ }
+ spec->multiout.hp_out_nid[0] = alc_auto_look_for_dac(codec, cfg->hp_pins[0]);
+ return 0;
+ }
+
/* fill hard-wired DACs first */
if (!redone) {
for (i = 0; i < cfg->line_outs; i++)
@@ -3126,6 +3158,7 @@ static int alc_auto_fill_dac_nids(struct
}
}
+ spec->alt_dac_nid = spec->private_dac_nids[3];
return 0;
}
@@ -3255,7 +3288,12 @@ static int alc_auto_create_multi_out_ctl
if (i >= cfg->line_outs)
pin = spec->multi_io[i - 1].pin;
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];
sw = alc_look_for_out_mute_nid(codec, pin, dac);
vol = alc_look_for_out_vol_nid(codec, pin, dac);
@@ -3275,9 +3313,11 @@ static int alc_auto_create_multi_out_ctl
if (err < 0)
return err;
} else {
+ 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;
@@ -3617,12 +3657,16 @@ static int alc_set_multi_io(struct hda_c
HDA_AMP_MUTE, 0);
alc_auto_select_dac(codec, nid, spec->multi_io[idx].dac);
} else {
- if (get_wcaps(codec, nid) & AC_WCAP_OUT_AMP)
+ if ( i < 2 ) {
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);
+ }
}
return 0;
}
@@ -3645,6 +3689,19 @@ static int alc_auto_ch_mode_put(struct s
spec->multiout.max_channels = spec->ext_channel_count;
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;
}
@@ -5850,6 +5907,7 @@ enum {
ALC662_FIXUP_ASUS_MODE6,
ALC662_FIXUP_ASUS_MODE7,
ALC662_FIXUP_ASUS_MODE8,
+ ALC892_FIXUP_CLEVO_4ST_8CH,
};
static const struct alc_fixup alc662_fixups[] = {
@@ -5995,6 +6053,20 @@ static const struct alc_fixup alc662_fix
.chained = true,
.chain_id = ALC662_FIXUP_SKU_IGNORE
},
+ [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 */
+ { }
+ },
+ },
};
static const struct snd_pci_quirk alc662_fixup_tbl[] = {
@@ -6008,6 +6080,8 @@ static const struct snd_pci_quirk alc662
SND_PCI_QUIRK(0x17aa, 0x38af, "Lenovo Ideapad Y550P", ALC662_FIXUP_IDEAPAD),
SND_PCI_QUIRK(0x17aa, 0x3a0d, "Lenovo Ideapad Y550", ALC662_FIXUP_IDEAPAD),
SND_PCI_QUIRK(0x1b35, 0x2206, "CZC P10T", ALC662_FIXUP_CZC_P10T),
+ SND_PCI_QUIRK(0x1558, 0x5102, "Clevo P150HM", ALC892_FIXUP_CLEVO_4ST_8CH),
+ SND_PCI_QUIRK(0x1558, 0x7100, "Clevo P170HM", ALC892_FIXUP_CLEVO_4ST_8CH),
#if 0
/* Below is a quirk table taken from the old code.
[-- Attachment #3: Type: text/plain, Size: 0 bytes --]
next prev parent reply other threads:[~2011-11-28 10:28 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-08-20 7:33 Clevo P170HM / Sager NP8170 audio Andrew Mahone
2011-11-08 5:42 ` Julian Sikorski
2011-11-16 13:08 ` Raymond Yau
2011-11-16 15:15 ` Julian Sikorski
2011-11-17 2:08 ` Raymond Yau
2011-11-17 8:46 ` Julian Sikorski
2011-11-17 8:55 ` Julian Sikorski
2011-11-19 2:31 ` Raymond Yau
2011-11-19 9:11 ` Julian Sikorski
2011-11-21 6:47 ` Raymond Yau
2011-11-21 11:39 ` Julian Sikorski
2011-11-22 5:17 ` Raymond Yau
2011-11-22 12:45 ` Julian Sikorski
2011-11-25 6:39 ` Raymond Yau
2011-11-25 12:10 ` Julian Sikorski
2011-11-28 0:42 ` Raymond Yau
2011-11-28 10:27 ` Julian Sikorski [this message]
2011-11-28 11:46 ` Raymond Yau
2011-11-28 12:02 ` Julian Sikorski
2011-11-28 12:34 ` Julian Sikorski
2011-11-28 23:45 ` Raymond Yau
2011-11-29 10:25 ` Julian Sikorski
2011-12-01 16:35 ` Julian Sikorski
2011-12-02 0:52 ` Raymond Yau
2011-12-02 15:41 ` Julian Sikorski
2011-12-03 23:59 ` Raymond Yau
2011-12-04 12:41 ` Julian Sikorski
2011-11-24 9:20 ` Julian Sikorski
2011-11-24 10:03 ` Julian Sikorski
2011-11-24 10:30 ` Julian Sikorski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='javnn0$m3$1@dough.gmane.org' \
--to=belegdol@gmail.com \
--cc=alsa-devel@alsa-project.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.