Re: model needed for Clevo P150HM (Realtek ALC892)

[Takashi Iwai <tiwai@suse.de>]

At Sat, 25 Jun 2011 14:01:06 +0200,
Julian Sikorski wrote:
> 
> W dniu 25.06.2011 13:31, Julian Sikorski pisze:
> > W dniu 25.06.2011 13:08, Julian Sikorski pisze:
> >> W dniu 25.06.2011 09:11, Takashi Iwai pisze:
> >>> At Fri, 24 Jun 2011 13:25:53 +0200,
> >>> Julian Sikorski wrote:
> >>>>
> >>>> On 24/06/2011 11:12, Takashi Iwai wrote:
> >>>>> At Fri, 24 Jun 2011 10:34:01 +0200,
> >>>>> Takashi Iwai wrote:
> >>>>>>
> >>>>>> At Thu, 23 Jun 2011 18:40:47 +0200,
> >>>>>> Julian Sikorski wrote:
> >>>>>>>
> >>>>>>> On 23/06/2011 14:41, Takashi Iwai wrote:
> >>>>>>>> At Thu, 23 Jun 2011 08:36:55 +0200,
> >>>>>>>> Julian Sikorski wrote:
> >>>>>>>>>
> >>>>>>>>> Hi,
> >>>>>>>>>
> >>>>>>>>> I am coming here from pulseaudio-general mailing list [1]. I recently
> >>>>>>>>> got myself a Clevo P150HM laptop. It has 2.1 speakers, a built-in
> >>>>>>>>> microphone and the following connectors:
> >>>>>>>>> - headphone out
> >>>>>>>>> - microphone in; can be switched to center/lfe out
> >>>>>>>>> - spdif out; can be switched to rear speaker out
> >>>>>>>>> - line in; can be switched to side speaker out
> >>>>>>>>> Alsa does not seem to reflect that the device is capable of 5.1 and 7.1
> >>>>>>>>> sound, and there are two configs showing up in pavucontrol (analog
> >>>>>>>>> output and analog headphones) which seem to be the same. I have uploaded
> >>>>>>>>> the output of alsa-info.sh [2]. Please let me know if more information
> >>>>>>>>> is needed.
> >>>>>>>>
> >>>>>>>> The current parser doesn't allow the multi-io pins when the primary
> >>>>>>>> output is no line-out.  The patch below should fix (change) the
> >>>>>>>> behavior.  It'll count HP out, too.
> >>>>>>>>
> >>>>>>>>
> >>>>>>>> Takashi
> >>>>>>>>
> >>>>>>>> ---
> >>>>>>>> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> >>>>>>>> index fe56b2b..b0cf726 100644
> >>>>>>>> --- a/sound/pci/hda/patch_realtek.c
> >>>>>>>> +++ b/sound/pci/hda/patch_realtek.c
> >>>>>>>> @@ -18992,6 +18992,7 @@ static int alc662_auto_fill_dac_nids(struct hda_codec *codec,
> >>>>>>>>    	hda_nid_t dac;
> >>>>>>>>
> >>>>>>>>    	spec->multiout.dac_nids = spec->private_dac_nids;
> >>>>>>>> +	spec->multiout.num_dacs = 0;
> >>>>>>>>    	for (i = 0; i<   cfg->line_outs; i++) {
> >>>>>>>>    		dac = alc_auto_look_for_dac(codec, cfg->line_out_pins[i]);
> >>>>>>>>    		if (!dac)
> >>>>>>>> @@ -19326,8 +19327,20 @@ static int alc_auto_add_multi_channel_mode(struct hda_codec *codec)
> >>>>>>>>    	unsigned int location, defcfg;
> >>>>>>>>    	int num_pins;
> >>>>>>>>
> >>>>>>>> +	if (cfg->line_out_type == AUTO_PIN_SPEAKER_OUT&&   cfg->hp_outs == 1) {
> >>>>>>>> +		/* use HP as primary out */
> >>>>>>>> +		cfg->speaker_outs = cfg->line_outs;
> >>>>>>>> +		memcpy(cfg->speaker_pins, cfg->line_out_pins,
> >>>>>>>> +		       sizeof(cfg->speaker_pins));
> >>>>>>>> +		cfg->line_outs = cfg->hp_outs;
> >>>>>>>> +		memcpy(cfg->line_out_pins, cfg->hp_pins, sizeof(cfg->hp_pins));
> >>>>>>>> +		cfg->hp_outs = 0;
> >>>>>>>> +		memset(cfg->hp_pins, 0, sizeof(cfg->hp_pins));
> >>>>>>>> +		cfg->line_out_type = AUTO_PIN_HP_OUT;
> >>>>>>>> +		alc662_auto_fill_dac_nids(codec, cfg);
> >>>>>>>> +	}
> >>>>>>>>    	if (cfg->line_outs != 1 ||
> >>>>>>>> -	    cfg->line_out_type != AUTO_PIN_LINE_OUT)
> >>>>>>>> +	    cfg->line_out_type == AUTO_PIN_SPEAKER_OUT)
> >>>>>>>>    		return 0;
> >>>>>>>>
> >>>>>>>>    	defcfg = snd_hda_codec_get_pincfg(codec, cfg->line_out_pins[0]);
> >>>>>>>> @@ -19348,6 +19361,8 @@ static int alc_auto_add_multi_channel_mode(struct hda_codec *codec)
> >>>>>>>>    		spec->multi_ios = num_pins;
> >>>>>>>>    		spec->ext_channel_count = 2;
> >>>>>>>>    		spec->multiout.num_dacs = num_pins + 1;
> >>>>>>>> +		/* for avoiding multi HP mixers */
> >>>>>>>> +		cfg->line_out_type = AUTO_PIN_LINE_OUT;
> >>>>>>>>    	}
> >>>>>>>>    	return 0;
> >>>>>>>>    }
> >>>>>>> Would you like me to test this patch? If so, what should I apply it to
> >>>>>>> and how should I proceed?
> >>>>>>
> >>>>>> Yes, the test would be appreciated, of course.
> >>>>>> Apply it to the latest 3.0-rc kernel and test.
> >>>>>
> >>>>> Also, the additional patch below will give "Headphone" mixer control
> >>>>> back while new "Surround", "Center" and "LFE" will appear.
> >>>>>
> >>>>> Note that you'll need to change "Channel Mode" mixer control to "6ch"
> >>>>> for using the surround outputs even with these patches.  As default,
> >>>>> it's set as "2ch".
> >>>>>
> >>>>>
> >>>>> Takashi
> >>>>>
> >>>> I will definitely try to give this a shot, but the outcome might be 
> >>>> dependent on the fact whether Fedora 15 will want to work with 3.0-rc 
> >>>> kernel.
> >>>
> >>> You can build the kernel by yourself at any time...
> >>>
> >>>> How about 8-channel sound btw? Under Windows the device claims to be 
> >>>> capable of it.
> >>>
> >>> Do you have 4 jacks?  Otherwise it's nonsense.
> >>>
> >>>
> >>> Takashi
> >> OK, I was able to successfully apply the patch and build the kernel. I
> >> tested it using:
> >> $ pasuspender bash
> >> $ speaker-test -Dplug:surround51 -c6
> >> Once I set sound to 6-channel using alsamixer -c0, I was able to hear
> >> the sound for all 6 channels. Setting it to 4 channels took away LFE and
> >> center. Pulseaudio outputs are still a little messy, but this is out of
> >> ALSA scope, I think.
> >>
> >> Julian
> > It seems I spoke a bit too soon. The superfluous controls are still a
> > problem. Speaker and Headphone controls seem to be overlapping (new
> > output on amixer -c0 attached). They both control the headphones when
> > they are plugged in and internal speakers when not.
> > 
> > Julian
> > 
> Sorry for this email bombardment, but I just booted back to stock F15
> kernel (2.6.38.8-32.fc15.x86_64) and noticed that speaker and headphone
> controls actually do what they advertise. Still, it is impossible to
> make the sound go through the speakers when headphones are plugged in (I
> tried pavucontrol for this).

The auto-mute feature can be controlled "Auto-Mute Mode" mixer enum.
Set to "Disabled" will turn off the auto-mute in the driver side.
However...

> The converse might be true though - if you
> select "analog headphones" in pavucontrols the speakers go silent.

This is what PA does intentionally, independently from the driver
auto-mute behavior.


Takashi