Re: mm snapshot broken-out-2007-03-05-02-22.tar.gz uploaded

From: Takashi Iwai
Date: Thu Mar 08 2007 - 10:55:45 EST


At Thu, 08 Mar 2007 16:42:26 +0100,
Michal Piotrowski wrote:
>
> Takashi Iwai napisał(a):
> > At Wed, 07 Mar 2007 23:22:36 +0100,
> > Michal Piotrowski wrote:
> >> Hi Takashi,
> >>
> >> Takashi Iwai napisał(a):
> >>> At Wed, 7 Mar 2007 11:50:24 -0800,
> >>> Andrew Morton wrote:
> >>>> On Wed, 7 Mar 2007 20:41:30 +0100 "Michal Piotrowski" <michal.k.k.piotrowski@xxxxxxxxx> wrote:
> >>>>
> >>>>>>> 831466f4ad2b5fe23dff77edbe6a7c244435e973 is first bad commit
> >>>>>>> commit 831466f4ad2b5fe23dff77edbe6a7c244435e973
> >>>>>>> Author: Randy Cushman <rcushman_linux@xxxxxxxxxxxxx>
> >>>>>>> Date: Tue Dec 19 18:42:16 2006 +0100
> >>>>>>>
> >>>>>>> [ALSA] ac97 - fix microphone and line_in selection logic
> >>>>>>>
> >>>>>>> This patch fixes the Microphone and LINE_IN select logic for
> >>>>>>> Analog Devices surround codecs with shared jacks. The existing
> >>>>>>> code can never utilize the shared jacks for Microphone and LINE_IN
> >>>>>>> due to the reversed jack selection logic. The patched code
> >>>>>>> correctly selects the shared jack for input if the 'Channel Mode'
> >>>>>>> selector does not specify that the jack is to be used for output.
> >>>>>>> Specifically, in '2ch' mode the Center/LFE jack is used for
> >>>>>>> microphone input and the Surround jack is used for LINE_IN,
> >>>>>>> in '4ch' mode the Center/LFE jack is used for microphone input
> >>>>>>> and the Surround jack is used for output, and in '6ch' mode
> >>>>>>> both jacks are used for output.
> >>>>>>>
> >>>>>>> Signed-off-by: Randy Cushman <rcushman_linux@xxxxxxxxxxxxx>
> >>>>>>> Signed-off-by: Takashi Iwai <tiwai@xxxxxxx>
> >>>>>>> Signed-off-by: Jaroslav Kysela <perex@xxxxxxx>
> >>>>>>>
> >>>>>> Is 2.6.21-rc3 similarly broken?
> >>>>> Yes.
> >>>>>
> >>>> OK. Can you please confirm that applying the below fixes 2.6.21-rc3?
> >>>>
> >>>> If so, I'll queue it up for application in a week or two if there is no
> >>>> progress on this.
> >>> [sorry for the late interrupt - I'm just back from a vacation]
> >>>
> >>> Could you postpone reverting for a while?
> >>> I guess this possible regression is limited only to a certain type of
> >>> ac97 codec chip. The patch applied there was a fix for many codec
> >>> chips.
> >>>
> >>> Which codec chip are you using specifically? Please check
> >>> /proc/asound/card0/codec97#0/* file(s).
> >> 0-0/0: Analog Devices AD1985
> >
> > OK, that's good to know.
> >
> >> control.43 {
> >> comment.access 'read write'
> >> comment.type BOOLEAN
> >> comment.count 1
> >> iface MIXER
> >> name 'Spread Front to Surround and Center/LFE'
> >> value true
> >> }
> >
> > So, you're playing two-channel samples and get no output from surround
> > and CLFE?
>
> Yes.
>
> > Or, does it happen if you play a real 5.1 channel file?
> > (for example, try speaker-test -c6)
>
> It doesn't work for me on both kernels.

Pass -Dplug:surround51 to speaker-test in addition.


Takashi
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/