From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julian Sikorski Subject: Re: model needed for Clevo P150HM (Realtek ALC892) Date: Wed, 29 Jun 2011 20:16:00 +0200 Message-ID: <4E0B6BE0.1010003@gmail.com> References: <4E036C8F.3080709@gmail.com> <4E047441.4010003@gmail.com> <4E05C1B1.50302@gmail.com> <4E05C704.4080806@gmail.com> <4E05CE02.1070801@gmail.com> <4E082166.5060902@gmail.com> <4E08E33D.3060608@gmail.com> <4E0B62B6.6000007@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ww0-f51.google.com (mail-ww0-f51.google.com [74.125.82.51]) by alsa0.perex.cz (Postfix) with ESMTP id 479591038F0 for ; Wed, 29 Jun 2011 20:16:06 +0200 (CEST) Received: by wwj26 with SMTP id 26so1155289wwj.20 for ; Wed, 29 Jun 2011 11:16:05 -0700 (PDT) In-Reply-To: <4E0B62B6.6000007@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: alsa-devel-bounces@alsa-project.org Errors-To: alsa-devel-bounces@alsa-project.org Cc: Takashi Iwai , alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org 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