Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Julian Sikorski <belegdol@gmail.com>
Cc: Takashi Iwai <tiwai@suse.de>, alsa-devel@alsa-project.org
Subject: Re: model needed for Clevo P150HM (Realtek ALC892)
Date: Wed, 29 Jun 2011 20:16:00 +0200	[thread overview]
Message-ID: <4E0B6BE0.1010003@gmail.com> (raw)
In-Reply-To: <4E0B62B6.6000007@gmail.com>

W dniu 29.06.2011 19:36, Julian Sikorski pisze:
> W dniu 28.06.2011 09:12, Takashi Iwai pisze:
>> At Mon, 27 Jun 2011 22:08:29 +0200,
>> Julian Sikorski wrote:
>>>
>>> W dniu 27.06.2011 14:32, Takashi Iwai pisze:
>>>> At Mon, 27 Jun 2011 11:19:36 +0200,
>>>> Takashi Iwai wrote:
>>>>>
>>>>> At Mon, 27 Jun 2011 08:21:26 +0200,
>>>>> Julian Sikorski wrote:
>>>>>>
>>>>>> W dniu 27.06.2011 08:04, Takashi Iwai pisze:
>>>>>>> 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...
>>>>>>
>>>>>> You are right - when auto-mute is disabled sound goes via both speakers
>>>>>> and headphones when they are plugged. I did not know the function of
>>>>>> auto-mute.
>>>>>> However, the "Speaker" and "Headphone" controls still do influence both
>>>>>> outputs, and "Speaker" can only be muted - there is no volume control
>>>>>> available. So, in my opinion, they should either be merged altogether,
>>>>>> or switched back to the way they work in 2.6.38.8-32.fc15.x86_64, that
>>>>>> is setting the volume of speakers and headphones independently.
>>>>>
>>>>> Give alsa-info.sh outputs on your machine with the latest status for
>>>>> further analysis.
>>>>
>>>> I think this issue was fixed now in topic/hda branch in sound git tree:
>>>>   git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound-2.6.git
>>>>
>>>> Give it a try.  You can pull the branch into the latest Linus git tree
>>>> for merging into the latest 3.0-rc.
>>>>
>>>>
>>>> Takashi
>>> It seems like the problem is still there. The output of alsa-info.sh is
>>> available here:
>>> http://www.alsa-project.org/db/?f=38e2e44ab86691006a837715601aad4311fda0d3
>>
>> Double-check whether you compiled really the right stuff.
>> The emulator works fine, so something wrong likely in your side.
>>
>>
>> Takashi
> I compiled the latest origin/master of sound-2.6 tree and the problem is
> still there. Output of alsa-info.sh is attached.
> 
> Julian
> 
I had a closer look at 2.6.38.8-32.fc15.x86_64 and the controls are not
completely separate. While you can regulate the volume of headphones and
speakers separately, muting one mutes the other.

Julian

  reply	other threads:[~2011-06-29 18:16 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-23  6:36 model needed for Clevo P150HM (Realtek ALC892) Julian Sikorski
2011-06-23 12:41 ` Takashi Iwai
2011-06-23 16:40   ` Julian Sikorski
2011-06-24  8:34     ` Takashi Iwai
2011-06-24  9:12       ` Takashi Iwai
2011-06-24 11:25         ` Julian Sikorski
2011-06-25  7:11           ` Takashi Iwai
2011-06-25  9:22             ` Julian Sikorski
2011-06-25 11:08             ` Julian Sikorski
2011-06-25 11:31               ` Julian Sikorski
2011-06-25 12:01                 ` Julian Sikorski
2011-06-27  6:04                   ` Takashi Iwai
2011-06-27  6:21                     ` Julian Sikorski
2011-06-27  9:19                       ` Takashi Iwai
2011-06-27 12:32                         ` Takashi Iwai
2011-06-27 12:42                           ` Julian Sikorski
2011-06-27 12:46                             ` Takashi Iwai
2011-06-27 12:55                               ` Julian Sikorski
2011-06-27 16:12                                 ` Takashi Iwai
2011-06-27 20:08                           ` Julian Sikorski
2011-06-28  7:12                             ` Takashi Iwai
2011-06-28  8:03                               ` Julian Sikorski
2011-06-28  9:05                                 ` Takashi Iwai
2011-06-28 10:36                                   ` Julian Sikorski
2011-06-28 12:40                                     ` Takashi Iwai
2011-06-29 17:36                               ` Julian Sikorski
2011-06-29 18:16                                 ` Julian Sikorski [this message]
2011-06-29 19:29                                   ` Julian Sikorski
2011-06-30  5:26                                     ` Takashi Iwai
2011-06-30  6:33                                       ` Julian Sikorski
     [not found]                                       ` <4E0C17F2.5070704@gmail.com>
2011-06-30  6:54                                         ` Takashi Iwai
2011-06-30 17:00                                           ` Julian Sikorski
2011-06-30 17:41                                             ` Takashi Iwai
2011-06-30 18:09                                               ` Julian Sikorski
2011-06-30 19:28                                                 ` Takashi Iwai
2011-06-30 20:16                                                   ` Julian Sikorski
2011-07-01  5:21                                                     ` Takashi Iwai
2011-07-01  6:14                                                       ` Julian Sikorski
2011-07-01  6:24                                                         ` Takashi Iwai
2011-07-01  6:27                                                           ` Julian Sikorski
2011-07-01  7:07                                                             ` Julian Sikorski
2011-07-01  7:14                                                               ` Takashi Iwai
2011-07-01 11:18                                                                 ` Julian Sikorski
2011-07-01 22:09                                                                 ` Julian Sikorski
2011-07-01 22:37                                                                   ` Julian Sikorski
2011-07-04 20:47                                                                     ` Julian Sikorski
2011-07-05  8:16                                                           ` Julian Sikorski
2011-07-01  7:32                                                       ` Takashi Iwai
2011-07-01 18:57                                                         ` Julian Sikorski
2011-07-01 19:25                                                           ` Takashi Iwai
2011-07-01 19:35                                                             ` Julian Sikorski
2011-07-03 11:22                                                       ` Julian Sikorski
2011-07-04 22:50                                                       ` Julian Sikorski
2011-07-05  1:31                                                         ` Raymond Yau
2011-07-05  7:45                                                           ` Julian Sikorski
2011-07-17 11:55                                                           ` Julian Sikorski
2011-07-17 12:17                                                             ` Julian Sikorski
2011-07-18  7:35                                                               ` Takashi Iwai
2011-07-18  8:52                                                                 ` Takashi Iwai
2011-07-18 11:15                                                                   ` Julian Sikorski
2011-07-18 11:18                                                                     ` Julian Sikorski
2011-07-18 11:19                                                                     ` Takashi Iwai
2011-07-18 11:42                                                                       ` Julian Sikorski
2011-07-18 12:16                                                                         ` Takashi Iwai
2011-07-18 12:42                                                                           ` Julian Sikorski
2011-07-18 13:23                                                                             ` Takashi Iwai
2011-07-18 13:36                                                                               ` Julian Sikorski
2011-07-05  5:32                                                         ` Takashi Iwai
  -- strict thread matches above, loose matches on Subject: below --
2011-06-23  6:40 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=4E0B6BE0.1010003@gmail.com \
    --to=belegdol@gmail.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=tiwai@suse.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox