From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: model needed for Clevo P150HM (Realtek ALC892) Date: Tue, 28 Jun 2011 09:12:59 +0200 Message-ID: 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> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mx2.suse.de (cantor2.suse.de [195.135.220.15]) by alsa0.perex.cz (Postfix) with ESMTP id E118B2443B for ; Tue, 28 Jun 2011 09:12:59 +0200 (CEST) In-Reply-To: <4E08E33D.3060608@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 To: Julian Sikorski Cc: alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org 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