From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932437AbXCHSvk (ORCPT ); Thu, 8 Mar 2007 13:51:40 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S933038AbXCHSvk (ORCPT ); Thu, 8 Mar 2007 13:51:40 -0500 Received: from mx2.suse.de ([195.135.220.15]:58466 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932437AbXCHSvj (ORCPT ); Thu, 8 Mar 2007 13:51:39 -0500 Date: Thu, 08 Mar 2007 19:51:36 +0100 Message-ID: From: Takashi Iwai To: Randy Cushman Cc: Michal Piotrowski , Andrew Morton , Jaroslav Kysela , alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org, Linus Torvalds , Adrian Bunk Subject: Re: mm snapshot broken-out-2007-03-05-02-22.tar.gz uploaded In-Reply-To: <45F0426D.6020408@earthlink.net> References: <200703051024.l25AOIF3015951@shell0.pdx.osdl.net> <45EEA3C4.7000501@googlemail.com> <20070307104746.f08a2e84.akpm@linux-foundation.org> <6bffcb0e0703071141j42309702w793e4a76b1b23bfa@mail.gmail.com> <20070307115024.756c848f.akpm@linux-foundation.org> <45EF3B2C.9060801@googlemail.com> <45F02EE2.6040606@googlemail.com> <6bffcb0e0703080852l4a06c0f9qeb51044a7a1a68a@mail.gmail.com> <45F0426D.6020408@earthlink.net> User-Agent: Wanderlust/2.12.0 (Your Wildest Dreams) SEMI/1.14.6 (Maruoka) FLIM/1.14.7 (=?ISO-8859-4?Q?Sanj=F2?=) APEL/10.6 MULE XEmacs/21.5 (beta27) (fiddleheads) (+CVS-20060704) (i386-suse-linux) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org At Thu, 08 Mar 2007 12:05:49 -0500, Randy Cushman wrote: > > Takashi Iwai wrote: > > At Thu, 8 Mar 2007 17:52:43 +0100, > > Michal Piotrowski wrote: > > > >> On 08/03/07, Takashi Iwai wrote: > >> > >>> At Thu, 08 Mar 2007 16:42:26 +0100, > >>> Michal Piotrowski wrote: > >>> > >>>>> Also, it'd be helpful if you compare > >>>>> /proc/asound/card0/codec97#0/ac97#0-0+regs file before and after the > >>>>> patch. This reveals which register bits differ actually. > >>>>> > >>>> a2.txt is form 2.6.21-rc3 witchout 831466f4ad2b5fe23dff77edbe6a7c244435e973 > >>>> > >>>> diff -uprN a2.txt /proc/asound/card0/codec97#0/ac97#0-0+regs > >>>> --- a2.txt 2007-03-08 16:36:47.000000000 +0100 > >>>> +++ /proc/asound/card0/codec97#0/ac97#0-0+regs 2007-03-08 16:36:49.447083597 +0100 > >>>> @@ -56,8 +56,8 @@ > >>>> 0:6e = 0000 > >>>> 0:70 = 0000 > >>>> 0:72 = 0004 > >>>> -0:74 = 1001 > >>>> -0:76 = 64a4 > >>>> +0:74 = 1201 > >>>> +0:76 = 7ca4 > >>>> 0:78 = 000c > >>>> 0:7a = 0000 > >>>> 0:7c = 4144 > >>>> > >>> OK, some wrong bits there. Could you try the patch below? > >>> > >> Problem fixed. Thanks! > >> > >> (speaker-test -c6 -Dplug:surround51 is still broken) > >> > > > > Hm, how about turning off "Spread Front To..." control and run > > speaker-test? > > > > > > Takashi > > > > > > > Michal and Takashi, > > I've been researching your issue this morning. I just noticed that your > exchange had ensued. > > I configured my AD1986-based system as closely as I could to match your > AD1985 mixer settings. In doing so, I experienced results similar to > what I'm assuming you described: Only front channel sound came out of > the surround and c/lfe jacks. > > Here is what I had to change to get the appropriate multichannel sound: > > 1) The command "speaker-test -c6" does not work on my system. I have to > use "speaker-test -Dsurround51 -c6" (Perhaps this is related to running > an old version of speaker-test.) > > 2) I disabled "Spread Front to Surround and Center/LFE" > > The explanation for 2) is this: There can be only one source for each > of the Surround and Center/LFE jacks active at one time. When "Spread > Front to Surround and Center/LFE" is disabled, the surround channels go > to the the jacks; when this control is enabled, ONLY the front channels > go to the surround and c/lfe jacks. > > Considering the register values Michal provided and Takashi's patch: > > The logic of the code is as follows: (Slightly simplifying,) functions > is_shared_linein() and is_shared_micin() return TRUE if the mixer is > configured for shared jacks AND NOT for 6 channel mode. In this case it > is appropriate to > set these bits (LODIS and CLDIS). Yep. So, can you confirm that my fix is OK? Then I'll queue it up to ALSA tree. thanks, Takashi