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

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

[Michal Piotrowski]

Hi,

akpm@linux-foundation.org napisał(a):
> The mm snapshot broken-out-2007-03-05-02-22.tar.gz has been uploaded to
> 
>    ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/mm/broken-out-2007-03-05-02-22.tar.gz
> 
> It contains the following patches against 2.6.21-rc2:
> 

Outstanding issue - my 3d surround doesn't work since 2.6.20.

I'll bisect this commits
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=724339d76d9407cd1a8ad32a9c1fdf64840cc51b
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=6026179519896e7d35b2564e7544487d1c8948e7

00:1f.5 Multimedia audio controller: Intel Corporation 82801EB/ER (ICH5/ICH5R) AC'97 Audio Controller (rev 02)
        Subsystem: ASUSTeK Computer Inc. P4P800 Mainboard
        Control: I/O+ Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B-
        Status: Cap+ 66MHz- UDF- FastB2B+ ParErr- DEVSEL=medium >TAbort- <TAbort- <MAbort- >SERR- <PERR-
        Latency: 0
        Interrupt: pin B routed to IRQ 17
        Region 0: I/O ports at d000 [size=256]
        Region 1: I/O ports at d400 [size=64]
        Region 2: Memory at f5fff800 (32-bit, non-prefetchable) [size=512]
        Region 3: Memory at f5fff400 (32-bit, non-prefetchable) [size=256]
        Capabilities: [50] Power Management version 2
                Flags: PMEClk- DSI- D1- D2- AuxCurrent=375mA PME(D0+,D1-,D2-,D3hot+,D3cold+)
                Status: D0 PME-Enable- DSel=0 DScale=0 PME-

Advanced Linux Sound Architecture Driver Version 1.0.14rc2 (Wed Feb 14 07:42:13 2007 UTC).

CONFIG_SND=m
CONFIG_SND_TIMER=m
CONFIG_SND_PCM=m
CONFIG_SND_RAWMIDI=m
CONFIG_SND_SEQUENCER=m
CONFIG_SND_SEQ_DUMMY=m
CONFIG_SND_OSSEMUL=y
CONFIG_SND_MIXER_OSS=m
CONFIG_SND_PCM_OSS=m
CONFIG_SND_PCM_OSS_PLUGINS=y
CONFIG_SND_SEQUENCER_OSS=y
CONFIG_SND_RTCTIMER=m
CONFIG_SND_SEQ_RTCTIMER_DEFAULT=y
# CONFIG_SND_DYNAMIC_MINORS is not set
CONFIG_SND_SUPPORT_OLD_API=y
CONFIG_SND_VERBOSE_PROCFS=y
# CONFIG_SND_VERBOSE_PRINTK is not set
# CONFIG_SND_DEBUG is not set

CONFIG_SND_AC97_CODEC=m
CONFIG_SND_DUMMY=m
CONFIG_SND_VIRMIDI=m

CONFIG_SND_INTEL8X0=m

Regards,
Michal

-- 
Michal K. K. Piotrowski
LTG - Linux Testers Group (PL)
(http://www.stardust.webpages.pl/ltg/)
LTG - Linux Testers Group (EN)
(http://www.stardust.webpages.pl/linux_testers_group_en/)

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Michal Piotrowski]

Michal Piotrowski napisał(a):
> Hi,
> 
> akpm@linux-foundation.org napisał(a):
>> The mm snapshot broken-out-2007-03-05-02-22.tar.gz has been uploaded to
>>
>>    ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/mm/broken-out-2007-03-05-02-22.tar.gz
>>
>> It contains the following patches against 2.6.21-rc2:
>>
> 
> Outstanding issue - my 3d surround doesn't work since 2.6.20.
> 
> I'll bisect this commits
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=724339d76d9407cd1a8ad32a9c1fdf64840cc51b
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=6026179519896e7d35b2564e7544487d1c8948e7
> 
> 00:1f.5 Multimedia audio controller: Intel Corporation 82801EB/ER (ICH5/ICH5R) AC'97 Audio Controller (rev 02)
>         Subsystem: ASUSTeK Computer Inc. P4P800 Mainboard
>         Control: I/O+ Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B-
>         Status: Cap+ 66MHz- UDF- FastB2B+ ParErr- DEVSEL=medium >TAbort- <TAbort- <MAbort- >SERR- <PERR-
>         Latency: 0
>         Interrupt: pin B routed to IRQ 17
>         Region 0: I/O ports at d000 [size=256]
>         Region 1: I/O ports at d400 [size=64]
>         Region 2: Memory at f5fff800 (32-bit, non-prefetchable) [size=512]
>         Region 3: Memory at f5fff400 (32-bit, non-prefetchable) [size=256]
>         Capabilities: [50] Power Management version 2
>                 Flags: PMEClk- DSI- D1- D2- AuxCurrent=375mA PME(D0+,D1-,D2-,D3hot+,D3cold+)
>                 Status: D0 PME-Enable- DSel=0 DScale=0 PME-
> 
> Advanced Linux Sound Architecture Driver Version 1.0.14rc2 (Wed Feb 14 07:42:13 2007 UTC).
> 
> CONFIG_SND=m
> CONFIG_SND_TIMER=m
> CONFIG_SND_PCM=m
> CONFIG_SND_RAWMIDI=m
> CONFIG_SND_SEQUENCER=m
> CONFIG_SND_SEQ_DUMMY=m
> CONFIG_SND_OSSEMUL=y
> CONFIG_SND_MIXER_OSS=m
> CONFIG_SND_PCM_OSS=m
> CONFIG_SND_PCM_OSS_PLUGINS=y
> CONFIG_SND_SEQUENCER_OSS=y
> CONFIG_SND_RTCTIMER=m
> CONFIG_SND_SEQ_RTCTIMER_DEFAULT=y
> # CONFIG_SND_DYNAMIC_MINORS is not set
> CONFIG_SND_SUPPORT_OLD_API=y
> CONFIG_SND_VERBOSE_PROCFS=y
> # CONFIG_SND_VERBOSE_PRINTK is not set
> # CONFIG_SND_DEBUG is not set
> 
> CONFIG_SND_AC97_CODEC=m
> CONFIG_SND_DUMMY=m
> CONFIG_SND_VIRMIDI=m
> 
> CONFIG_SND_INTEL8X0=m

As I said above, center and rear speakers doesn't work with this patch.

$ git-bisect good
831466f4ad2b5fe23dff77edbe6a7c244435e973 is first bad commit
commit 831466f4ad2b5fe23dff77edbe6a7c244435e973
Author: Randy Cushman <rcushman_linux@earthlink.net>
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@earthlink.net>
    Signed-off-by: Takashi Iwai <tiwai@suse.de>
    Signed-off-by: Jaroslav Kysela <perex@suse.cz>

:040000 040000 7146a2c5350578fe1b05586c64df99889fa423fe 10e98a9b4819b34ce2abb2c36adbf269d39b9e4c M      sound

Regards,
Michal

-- 
Michal K. K. Piotrowski
LTG - Linux Testers Group (PL)
(http://www.stardust.webpages.pl/ltg/)
LTG - Linux Testers Group (EN)
(http://www.stardust.webpages.pl/linux_testers_group_en/)

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Andrew Morton]

On Wed, 07 Mar 2007 12:36:36 +0100 Michal Piotrowski <michal.k.k.piotrowski@gmail.com> wrote:

> Michal Piotrowski napisał(a):
> > Hi,
> > 
> > akpm@linux-foundation.org napisał(a):
> >> The mm snapshot broken-out-2007-03-05-02-22.tar.gz has been uploaded to
> >>
> >>    ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/mm/broken-out-2007-03-05-02-22.tar.gz
> >>
> >> It contains the following patches against 2.6.21-rc2:
> >>
> > 
> > Outstanding issue - my 3d surround doesn't work since 2.6.20.
> > 
> > I'll bisect this commits
> > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=724339d76d9407cd1a8ad32a9c1fdf64840cc51b
> > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=6026179519896e7d35b2564e7544487d1c8948e7
> > 
> > 00:1f.5 Multimedia audio controller: Intel Corporation 82801EB/ER (ICH5/ICH5R) AC'97 Audio Controller (rev 02)
> >         Subsystem: ASUSTeK Computer Inc. P4P800 Mainboard
> >         Control: I/O+ Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B-
> >         Status: Cap+ 66MHz- UDF- FastB2B+ ParErr- DEVSEL=medium >TAbort- <TAbort- <MAbort- >SERR- <PERR-
> >         Latency: 0
> >         Interrupt: pin B routed to IRQ 17
> >         Region 0: I/O ports at d000 [size=256]
> >         Region 1: I/O ports at d400 [size=64]
> >         Region 2: Memory at f5fff800 (32-bit, non-prefetchable) [size=512]
> >         Region 3: Memory at f5fff400 (32-bit, non-prefetchable) [size=256]
> >         Capabilities: [50] Power Management version 2
> >                 Flags: PMEClk- DSI- D1- D2- AuxCurrent=375mA PME(D0+,D1-,D2-,D3hot+,D3cold+)
> >                 Status: D0 PME-Enable- DSel=0 DScale=0 PME-
> > 
> > Advanced Linux Sound Architecture Driver Version 1.0.14rc2 (Wed Feb 14 07:42:13 2007 UTC).
> > 
> > CONFIG_SND=m
> > CONFIG_SND_TIMER=m
> > CONFIG_SND_PCM=m
> > CONFIG_SND_RAWMIDI=m
> > CONFIG_SND_SEQUENCER=m
> > CONFIG_SND_SEQ_DUMMY=m
> > CONFIG_SND_OSSEMUL=y
> > CONFIG_SND_MIXER_OSS=m
> > CONFIG_SND_PCM_OSS=m
> > CONFIG_SND_PCM_OSS_PLUGINS=y
> > CONFIG_SND_SEQUENCER_OSS=y
> > CONFIG_SND_RTCTIMER=m
> > CONFIG_SND_SEQ_RTCTIMER_DEFAULT=y
> > # CONFIG_SND_DYNAMIC_MINORS is not set
> > CONFIG_SND_SUPPORT_OLD_API=y
> > CONFIG_SND_VERBOSE_PROCFS=y
> > # CONFIG_SND_VERBOSE_PRINTK is not set
> > # CONFIG_SND_DEBUG is not set
> > 
> > CONFIG_SND_AC97_CODEC=m
> > CONFIG_SND_DUMMY=m
> > CONFIG_SND_VIRMIDI=m
> > 
> > CONFIG_SND_INTEL8X0=m
> 
> As I said above, center and rear speakers doesn't work with this patch.
> 
> $ git-bisect good
> 831466f4ad2b5fe23dff77edbe6a7c244435e973 is first bad commit
> commit 831466f4ad2b5fe23dff77edbe6a7c244435e973
> Author: Randy Cushman <rcushman_linux@earthlink.net>
> 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@earthlink.net>
>     Signed-off-by: Takashi Iwai <tiwai@suse.de>
>     Signed-off-by: Jaroslav Kysela <perex@suse.cz>
> 

Is 2.6.21-rc3 similarly broken?

This patch reverts cleanly if needed. 

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Alsa-devel mailing list
Alsa-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/alsa-devel

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

[Michal Piotrowski]

On 07/03/07, Andrew Morton <akpm@linux-foundation.org> wrote:
> On Wed, 07 Mar 2007 12:36:36 +0100 Michal Piotrowski <michal.k.k.piotrowski@gmail.com> wrote:
>
> > Michal Piotrowski napisał(a):
> > > Hi,
> > >
> > > akpm@linux-foundation.org napisał(a):
> > >> The mm snapshot broken-out-2007-03-05-02-22.tar.gz has been uploaded to
> > >>
> > >>    ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/mm/broken-out-2007-03-05-02-22.tar.gz
> > >>
> > >> It contains the following patches against 2.6.21-rc2:
> > >>
> > >
> > > Outstanding issue - my 3d surround doesn't work since 2.6.20.
> > >
> > > I'll bisect this commits
> > > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=724339d76d9407cd1a8ad32a9c1fdf64840cc51b
> > > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=6026179519896e7d35b2564e7544487d1c8948e7
> > >
> > > 00:1f.5 Multimedia audio controller: Intel Corporation 82801EB/ER (ICH5/ICH5R) AC'97 Audio Controller (rev 02)
> > >         Subsystem: ASUSTeK Computer Inc. P4P800 Mainboard
> > >         Control: I/O+ Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B-
> > >         Status: Cap+ 66MHz- UDF- FastB2B+ ParErr- DEVSEL=medium >TAbort- <TAbort- <MAbort- >SERR- <PERR-
> > >         Latency: 0
> > >         Interrupt: pin B routed to IRQ 17
> > >         Region 0: I/O ports at d000 [size=256]
> > >         Region 1: I/O ports at d400 [size=64]
> > >         Region 2: Memory at f5fff800 (32-bit, non-prefetchable) [size=512]
> > >         Region 3: Memory at f5fff400 (32-bit, non-prefetchable) [size=256]
> > >         Capabilities: [50] Power Management version 2
> > >                 Flags: PMEClk- DSI- D1- D2- AuxCurrent=375mA PME(D0+,D1-,D2-,D3hot+,D3cold+)
> > >                 Status: D0 PME-Enable- DSel=0 DScale=0 PME-
> > >
> > > Advanced Linux Sound Architecture Driver Version 1.0.14rc2 (Wed Feb 14 07:42:13 2007 UTC).
> > >
> > > CONFIG_SND=m
> > > CONFIG_SND_TIMER=m
> > > CONFIG_SND_PCM=m
> > > CONFIG_SND_RAWMIDI=m
> > > CONFIG_SND_SEQUENCER=m
> > > CONFIG_SND_SEQ_DUMMY=m
> > > CONFIG_SND_OSSEMUL=y
> > > CONFIG_SND_MIXER_OSS=m
> > > CONFIG_SND_PCM_OSS=m
> > > CONFIG_SND_PCM_OSS_PLUGINS=y
> > > CONFIG_SND_SEQUENCER_OSS=y
> > > CONFIG_SND_RTCTIMER=m
> > > CONFIG_SND_SEQ_RTCTIMER_DEFAULT=y
> > > # CONFIG_SND_DYNAMIC_MINORS is not set
> > > CONFIG_SND_SUPPORT_OLD_API=y
> > > CONFIG_SND_VERBOSE_PROCFS=y
> > > # CONFIG_SND_VERBOSE_PRINTK is not set
> > > # CONFIG_SND_DEBUG is not set
> > >
> > > CONFIG_SND_AC97_CODEC=m
> > > CONFIG_SND_DUMMY=m
> > > CONFIG_SND_VIRMIDI=m
> > >
> > > CONFIG_SND_INTEL8X0=m
> >
> > As I said above, center and rear speakers doesn't work with this patch.
> >
> > $ git-bisect good
> > 831466f4ad2b5fe23dff77edbe6a7c244435e973 is first bad commit
> > commit 831466f4ad2b5fe23dff77edbe6a7c244435e973
> > Author: Randy Cushman <rcushman_linux@earthlink.net>
> > 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@earthlink.net>
> >     Signed-off-by: Takashi Iwai <tiwai@suse.de>
> >     Signed-off-by: Jaroslav Kysela <perex@suse.cz>
> >
>
> Is 2.6.21-rc3 similarly broken?

Yes.

>
> This patch reverts cleanly if needed.
>

Regards,
Michal

-- 
Michal K. K. Piotrowski
LTG - Linux Testers Group (PL)
(http://www.stardust.webpages.pl/ltg/)
LTG - Linux Testers Group (EN)
(http://www.stardust.webpages.pl/linux_testers_group_en/)
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Andrew Morton]

On Wed, 7 Mar 2007 20:41:30 +0100 "Michal Piotrowski" <michal.k.k.piotrowski@gmail.com> wrote:

> > > 831466f4ad2b5fe23dff77edbe6a7c244435e973 is first bad commit
> > > commit 831466f4ad2b5fe23dff77edbe6a7c244435e973
> > > Author: Randy Cushman <rcushman_linux@earthlink.net>
> > > 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@earthlink.net>
> > >     Signed-off-by: Takashi Iwai <tiwai@suse.de>
> > >     Signed-off-by: Jaroslav Kysela <perex@suse.cz>
> > >
> >
> > 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.




From: Andrew Morton <akpm@linux-foundation.org>

Revert 831466f4ad2b5fe23dff77edbe6a7c244435e973: "Michal Piotrowski"
<michal.k.k.piotrowski@gmail.com> reports that it broke his 3d surround.

Cc:"Michal Piotrowski" <michal.k.k.piotrowski@gmail.com>
Cc: Randy Cushman <rcushman_linux@earthlink.net>
Cc: Takashi Iwai <tiwai@suse.de>
Cc: Jaroslav Kysela <perex@suse.cz>
Cc: Adrian Bunk <bunk@stusta.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---

 sound/pci/ac97/ac97_patch.c |   66 +++++++++++++---------------------
 1 file changed, 26 insertions(+), 40 deletions(-)

diff -puN sound/pci/ac97/ac97_patch.c~revert-ac97-fix-microphone-and-line_in-selection-logic sound/pci/ac97/ac97_patch.c
--- a/sound/pci/ac97/ac97_patch.c~revert-ac97-fix-microphone-and-line_in-selection-logic
+++ a/sound/pci/ac97/ac97_patch.c
@@ -178,28 +178,14 @@ static inline int is_clfe_on(struct snd_
 	return ac97->channel_mode >= 2;
 }
 
-/* system has shared jacks with surround out enabled */
-static inline int is_shared_surrout(struct snd_ac97 *ac97)
-{
-	return !ac97->indep_surround && is_surround_on(ac97);
-}
-
-/* system has shared jacks with center/lfe out enabled */
-static inline int is_shared_clfeout(struct snd_ac97 *ac97)
-{
-	return !ac97->indep_surround && is_clfe_on(ac97);
-}
-
-/* system has shared jacks with line in enabled */
 static inline int is_shared_linein(struct snd_ac97 *ac97)
 {
-	return !ac97->indep_surround && !is_surround_on(ac97);
+	return ! ac97->indep_surround && is_surround_on(ac97);
 }
 
-/* system has shared jacks with mic in enabled */
 static inline int is_shared_micin(struct snd_ac97 *ac97)
 {
-	return !ac97->indep_surround && !is_clfe_on(ac97);
+	return ! ac97->indep_surround && is_clfe_on(ac97);
 }
 
 
@@ -2473,12 +2459,12 @@ static void alc650_update_jacks(struct s
 {
 	int shared;
 	
-	/* shared Line-In / Surround Out */
-	shared = is_shared_surrout(ac97);
+	/* shared Line-In */
+	shared = is_shared_linein(ac97);
 	snd_ac97_update_bits(ac97, AC97_ALC650_MULTICH, 1 << 9,
 			     shared ? (1 << 9) : 0);
-	/* update shared Mic In / Center/LFE Out */
-	shared = is_shared_clfeout(ac97);
+	/* update shared Mic */
+	shared = is_shared_micin(ac97);
 	/* disable/enable vref */
 	snd_ac97_update_bits(ac97, AC97_ALC650_CLOCK, 1 << 12,
 			     shared ? (1 << 12) : 0);
@@ -2608,12 +2594,12 @@ static void alc655_update_jacks(struct s
 {
 	int shared;
 	
-	/* shared Line-In / Surround Out */
-	shared = is_shared_surrout(ac97);
+	/* shared Line-In */
+	shared = is_shared_linein(ac97);
 	ac97_update_bits_page(ac97, AC97_ALC650_MULTICH, 1 << 9,
 			      shared ? (1 << 9) : 0, 0);
-	/* update shared Mic In / Center/LFE Out */
-	shared = is_shared_clfeout(ac97);
+	/* update shared mic */
+	shared = is_shared_micin(ac97);
 	/* misc control; vrefout disable */
 	snd_ac97_update_bits(ac97, AC97_ALC650_CLOCK, 1 << 12,
 			     shared ? (1 << 12) : 0);
@@ -2757,16 +2743,16 @@ static void alc850_update_jacks(struct s
 {
 	int shared;
 	
-	/* shared Line-In / Surround Out */
-	shared = is_shared_surrout(ac97);
+	/* shared Line-In */
+	shared = is_shared_linein(ac97);
 	/* SURR 1kOhm (bit4), Amp (bit5) */
 	snd_ac97_update_bits(ac97, AC97_ALC850_MISC1, (1<<4)|(1<<5),
 			     shared ? (1<<5) : (1<<4));
 	/* LINE-IN = 0, SURROUND = 2 */
 	snd_ac97_update_bits(ac97, AC97_ALC850_JACK_SELECT, 7 << 12,
 			     shared ? (2<<12) : (0<<12));
-	/* update shared Mic In / Center/LFE Out */
-	shared = is_shared_clfeout(ac97);
+	/* update shared mic */
+	shared = is_shared_micin(ac97);
 	/* Vref disable (bit12), 1kOhm (bit13) */
 	snd_ac97_update_bits(ac97, AC97_ALC850_MISC1, (1<<12)|(1<<13),
 			     shared ? (1<<12) : (1<<13));
@@ -2839,9 +2825,9 @@ static int patch_alc850(struct snd_ac97 
  */
 static void cm9738_update_jacks(struct snd_ac97 *ac97)
 {
-	/* shared Line-In / Surround Out */
+	/* shared Line-In */
 	snd_ac97_update_bits(ac97, AC97_CM9738_VENDOR_CTRL, 1 << 10,
-			     is_shared_surrout(ac97) ? (1 << 10) : 0);
+			     is_shared_linein(ac97) ? (1 << 10) : 0);
 }
 
 static const struct snd_kcontrol_new snd_ac97_cm9738_controls[] = {
@@ -2923,12 +2909,12 @@ static const struct snd_kcontrol_new snd
 
 static void cm9739_update_jacks(struct snd_ac97 *ac97)
 {
-	/* shared Line-In / Surround Out */
+	/* shared Line-In */
 	snd_ac97_update_bits(ac97, AC97_CM9739_MULTI_CHAN, 1 << 10,
-			     is_shared_surrout(ac97) ? (1 << 10) : 0);
-	/* shared Mic In / Center/LFE Out **/
+			     is_shared_linein(ac97) ? (1 << 10) : 0);
+	/* shared Mic */
 	snd_ac97_update_bits(ac97, AC97_CM9739_MULTI_CHAN, 0x3000,
-			     is_shared_clfeout(ac97) ? 0x1000 : 0x2000);
+			     is_shared_micin(ac97) ? 0x1000 : 0x2000);
 }
 
 static const struct snd_kcontrol_new snd_ac97_cm9739_controls[] = {
@@ -3040,8 +3026,8 @@ static void cm9761_update_jacks(struct s
 
 	val |= surr_on[ac97->spec.dev_flags][is_surround_on(ac97)];
 	val |= clfe_on[ac97->spec.dev_flags][is_clfe_on(ac97)];
-	val |= surr_shared[ac97->spec.dev_flags][is_shared_surrout(ac97)];
-	val |= clfe_shared[ac97->spec.dev_flags][is_shared_clfeout(ac97)];
+	val |= surr_shared[ac97->spec.dev_flags][is_shared_linein(ac97)];
+	val |= clfe_shared[ac97->spec.dev_flags][is_shared_micin(ac97)];
 
 	snd_ac97_update_bits(ac97, AC97_CM9761_MULTI_CHAN, 0x3c88, val);
 }
@@ -3289,12 +3275,12 @@ static int patch_vt1617a(struct snd_ac97
  */
 static void it2646_update_jacks(struct snd_ac97 *ac97)
 {
-	/* shared Line-In / Surround Out */
+	/* shared Line-In */
 	snd_ac97_update_bits(ac97, 0x76, 1 << 9,
-			     is_shared_surrout(ac97) ? (1<<9) : 0);
-	/* shared Mic / Center/LFE Out */
+			     is_shared_linein(ac97) ? (1<<9) : 0);
+	/* shared Mic */
 	snd_ac97_update_bits(ac97, 0x76, 1 << 10,
-			     is_shared_clfeout(ac97) ? (1<<10) : 0);
+			     is_shared_micin(ac97) ? (1<<10) : 0);
 }
 
 static const struct snd_kcontrol_new snd_ac97_controls_it2646[] = {
_


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Takashi Iwai]

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@gmail.com> wrote:
> 
> > > > 831466f4ad2b5fe23dff77edbe6a7c244435e973 is first bad commit
> > > > commit 831466f4ad2b5fe23dff77edbe6a7c244435e973
> > > > Author: Randy Cushman <rcushman_linux@earthlink.net>
> > > > 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@earthlink.net>
> > > >     Signed-off-by: Takashi Iwai <tiwai@suse.de>
> > > >     Signed-off-by: Jaroslav Kysela <perex@suse.cz>
> > > >
> > >
> > > 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).
Also, run "alsactl -f somefile store" and attach the generated file to
see whether any mixer status inconsistency exists.


Thanks,

Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Michal Piotrowski]

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@gmail.com> wrote:
>>
>>>>> 831466f4ad2b5fe23dff77edbe6a7c244435e973 is first bad commit
>>>>> commit 831466f4ad2b5fe23dff77edbe6a7c244435e973
>>>>> Author: Randy Cushman <rcushman_linux@earthlink.net>
>>>>> 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@earthlink.net>
>>>>>     Signed-off-by: Takashi Iwai <tiwai@suse.de>
>>>>>     Signed-off-by: Jaroslav Kysela <perex@suse.cz>
>>>>>
>>>> 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

PCI Subsys Vendor: 0x1043
PCI Subsys Device: 0x80f3

Revision         : 0x03
Compat. Class    : 0x00
Subsys. Vendor ID: 0xffff
Subsys. ID       : 0xffff

Capabilities     : -headphone out-
DAC resolution   : 20-bit
ADC resolution   : 16-bit
3D enhancement   : No 3D Stereo Enhancement

Current setup
Mic gain         : +20dB [+20dB]
POP path         : pre 3D
Sim. stereo      : off
3D enhancement   : off
Loudness         : off
Mono output      : MIX
Mic select       : Mic1
ADC/DAC loopback : off
Double rate slots: 10/11
Extended ID      : codec=0 rev=0 AMAP LDAC SDAC CDAC DSA=0 SPDIF DRA VRA
Extended status  : SPCV LDAC SDAC CDAC SPDIF=10/11 VRA
PCM front DAC    : 48000Hz
PCM Surr DAC     : 48000Hz
PCM LFE DAC      : 48000Hz
PCM ADC          : 48000Hz
SPDIF Control    : Consumer PCM Category=0x2 Generation=1 Rate=48kHz

                    Gain     Inverted  Buffer delay  Location
Master Out       :   0.0 dBV    -      16/fs         Rear I/O Panel
AUX Out          :   0.0 dBV    -      16/fs         Rear I/O Panel
Center/LFE Out   :   0.0 dBV    -      16/fs         Rear I/O Panel
Mic 1            :   0.0 dBV    -      16/fs         Rear I/O Panel
Mic 2            :   0.0 dBV    -      16/fs         Rear I/O Panel
Line In          :   0.0 dBV    -      16/fs         Rear I/O Panel



AD18XX configuration
Unchained        : 0x1000,0x0000,0x0000
Chained          : 0x0000,0x0000,0x0000


> Also, run "alsactl -f somefile store" and attach the generated file to
> see whether any mixer status inconsistency exists.

state.ICH5 {
	control.1 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 2
		iface MIXER
		name 'Master Surround Playback Switch'
		value.0 true
		value.1 true
	}
	control.2 {
		comment.access 'read write'
		comment.type INTEGER
		comment.count 2
		comment.range '0 - 31'
		iface MIXER
		name 'Master Surround Playback Volume'
		value.0 28
		value.1 28
	}
	control.3 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 1
		iface MIXER
		name 'Center Playback Switch'
		value true
	}
	control.4 {
		comment.access 'read write'
		comment.type INTEGER
		comment.count 1
		comment.range '0 - 31'
		iface MIXER
		name 'Center Playback Volume'
		value 28
	}
	control.5 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 1
		iface MIXER
		name 'LFE Playback Switch'
		value false
	}
	control.6 {
		comment.access 'read write'
		comment.type INTEGER
		comment.count 1
		comment.range '0 - 31'
		iface MIXER
		name 'LFE Playback Volume'
		value 0
	}
	control.7 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 2
		iface MIXER
		name 'Surround Playback Switch'
		value.0 true
		value.1 true
	}
	control.8 {
		comment.access 'read write'
		comment.type INTEGER
		comment.count 2
		comment.range '0 - 31'
		iface MIXER
		name 'Surround Playback Volume'
		value.0 28
		value.1 28
	}
	control.9 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 2
		iface MIXER
		name 'Master Playback Switch'
		value.0 true
		value.1 true
	}
	control.10 {
		comment.access 'read write'
		comment.type INTEGER
		comment.count 2
		comment.range '0 - 31'
		iface MIXER
		name 'Master Playback Volume'
		value.0 28
		value.1 28
	}
	control.11 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 1
		iface MIXER
		name 'Master Mono Playback Switch'
		value true
	}
	control.12 {
		comment.access 'read write'
		comment.type INTEGER
		comment.count 1
		comment.range '0 - 31'
		iface MIXER
		name 'Master Mono Playback Volume'
		value 28
	}
	control.13 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 1
		iface MIXER
		name 'PC Speaker Playback Switch'
		value false
	}
	control.14 {
		comment.access 'read write'
		comment.type INTEGER
		comment.count 1
		comment.range '0 - 15'
		iface MIXER
		name 'PC Speaker Playback Volume'
		value 0
	}
	control.15 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 1
		iface MIXER
		name 'Phone Playback Switch'
		value true
	}
	control.16 {
		comment.access 'read write'
		comment.type INTEGER
		comment.count 1
		comment.range '0 - 31'
		iface MIXER
		name 'Phone Playback Volume'
		value 29
	}
	control.17 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 1
		iface MIXER
		name 'Mic Playback Switch'
		value true
	}
	control.18 {
		comment.access 'read write'
		comment.type INTEGER
		comment.count 1
		comment.range '0 - 31'
		iface MIXER
		name 'Mic Playback Volume'
		value 29
	}
	control.19 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 1
		iface MIXER
		name 'Mic Boost (+20dB)'
		value true
	}
	control.20 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 2
		iface MIXER
		name 'Line Playback Switch'
		value.0 false
		value.1 false
	}
	control.21 {
		comment.access 'read write'
		comment.type INTEGER
		comment.count 2
		comment.range '0 - 31'
		iface MIXER
		name 'Line Playback Volume'
		value.0 0
		value.1 0
	}
	control.22 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 2
		iface MIXER
		name 'CD Playback Switch'
		value.0 true
		value.1 true
	}
	control.23 {
		comment.access 'read write'
		comment.type INTEGER
		comment.count 2
		comment.range '0 - 31'
		iface MIXER
		name 'CD Playback Volume'
		value.0 28
		value.1 28
	}
	control.24 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 2
		iface MIXER
		name 'Aux Playback Switch'
		value.0 true
		value.1 true
	}
	control.25 {
		comment.access 'read write'
		comment.type INTEGER
		comment.count 2
		comment.range '0 - 31'
		iface MIXER
		name 'Aux Playback Volume'
		value.0 30
		value.1 30
	}
	control.26 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 2
		iface MIXER
		name 'PCM Playback Switch'
		value.0 true
		value.1 true
	}
	control.27 {
		comment.access 'read write'
		comment.type INTEGER
		comment.count 2
		comment.range '0 - 31'
		iface MIXER
		name 'PCM Playback Volume'
		value.0 28
		value.1 28
	}
	control.28 {
		comment.access 'read write'
		comment.type ENUMERATED
		comment.count 2
		comment.item.0 Mic
		comment.item.1 CD
		comment.item.2 Video
		comment.item.3 Aux
		comment.item.4 Line
		comment.item.5 Mix
		comment.item.6 'Mix Mono'
		comment.item.7 Phone
		iface MIXER
		name 'Capture Source'
		value.0 CD
		value.1 CD
	}
	control.29 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 2
		iface MIXER
		name 'Capture Switch'
		value.0 true
		value.1 true
	}
	control.30 {
		comment.access 'read write'
		comment.type INTEGER
		comment.count 2
		comment.range '0 - 15'
		iface MIXER
		name 'Capture Volume'
		value.0 0
		value.1 0
	}
	control.31 {
		comment.access 'read write'
		comment.type ENUMERATED
		comment.count 1
		comment.item.0 Mix
		comment.item.1 Mic
		iface MIXER
		name 'Mono Output Select'
		value Mix
	}
	control.32 {
		comment.access 'read write'
		comment.type ENUMERATED
		comment.count 1
		comment.item.0 Mic1
		comment.item.1 Mic2
		iface MIXER
		name 'Mic Select'
		value Mic1
	}
	control.33 {
		comment.access read
		comment.type IEC958
		comment.count 1
		iface MIXER
		name 'IEC958 Playback Con Mask'
		value '0fff000f00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000'
	}
	control.34 {
		comment.access read
		comment.type IEC958
		comment.count 1
		iface MIXER
		name 'IEC958 Playback Pro Mask'
		value cf00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000
	}
	control.35 {
		comment.access 'read write'
		comment.type IEC958
		comment.count 1
		iface MIXER
		name 'IEC958 Playback Default'
		value '0082000200000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000'
	}
	control.36 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 1
		iface MIXER
		name 'IEC958 Playback Switch'
		value false
	}
	control.37 {
		comment.access 'read write'
		comment.type INTEGER
		comment.count 1
		comment.range '0 - 3'
		iface MIXER
		name 'IEC958 Playback AC97-SPSA'
		value 3
	}
	control.38 {
		comment.access 'read write'
		comment.type ENUMERATED
		comment.count 1
		comment.item.0 AC-Link
		comment.item.1 'A/D Converter'
		iface MIXER
		name 'IEC958 Playback Source'
		value AC-Link
	}
	control.39 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 1
		iface MIXER
		name 'Stereo Mic'
		value false
	}
	control.40 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 1
		iface MIXER
		name 'Exchange Center/LFE'
		value false
	}
	control.41 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 1
		iface MIXER
		name 'Exchange Front/Surround'
		value false
	}
	control.42 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 1
		iface MIXER
		name 'High Pass Filter Enable'
		value true
	}
	control.43 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 1
		iface MIXER
		name 'Spread Front to Surround and Center/LFE'
		value true
	}
	control.44 {
		comment.access 'read write'
		comment.type ENUMERATED
		comment.count 1
		comment.item.0 Off
		comment.item.1 '6 -> 4'
		comment.item.2 '6 -> 2'
		iface MIXER
		name Downmix
		value Off
	}
	control.45 {
		comment.access 'read write'
		comment.type ENUMERATED
		comment.count 1
		comment.item.0 High-Z
		comment.item.1 '3.7 V'
		comment.item.2 '2.25 V'
		comment.item.3 '0 V'
		iface MIXER
		name V_REFOUT
		value High-Z
	}
	control.46 {
		comment.access 'read write'
		comment.type ENUMERATED
		comment.count 1
		comment.item.0 Shared
		comment.item.1 Independent
		iface MIXER
		name 'Surround Jack Mode'
		value Shared
	}
	control.47 {
		comment.access 'read write'
		comment.type ENUMERATED
		comment.count 1
		comment.item.0 '2ch'
		comment.item.1 '4ch'
		comment.item.2 '6ch'
		iface MIXER
		name 'Channel Mode'
		value '6ch'
	}
	control.48 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 1
		iface MIXER
		name 'Headphone Jack Sense'
		value false
	}
	control.49 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 1
		iface MIXER
		name 'Line Jack Sense'
		value false
	}
	control.50 {
		comment.access 'read write'
		comment.type BOOLEAN
		comment.count 1
		iface MIXER
		name 'External Amplifier'
		value true
	}
}


> 
> 
> Thanks,
> 
> Takashi
> 

Regards,
Michal

-- 
Michal K. K. Piotrowski
LTG - Linux Testers Group (PL)
(http://www.stardust.webpages.pl/ltg/)
LTG - Linux Testers Group (EN)
(http://www.stardust.webpages.pl/linux_testers_group_en/)

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Alsa-devel mailing list
Alsa-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/alsa-devel

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

[Takashi Iwai]

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@gmail.com> wrote:
> >>
> >>>>> 831466f4ad2b5fe23dff77edbe6a7c244435e973 is first bad commit
> >>>>> commit 831466f4ad2b5fe23dff77edbe6a7c244435e973
> >>>>> Author: Randy Cushman <rcushman_linux@earthlink.net>
> >>>>> 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@earthlink.net>
> >>>>>     Signed-off-by: Takashi Iwai <tiwai@suse.de>
> >>>>>     Signed-off-by: Jaroslav Kysela <perex@suse.cz>
> >>>>>
> >>>> 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?  Or, does it happen if you play a real 5.1 channel file?
(for example, try speaker-test -c6)

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.


thanks,

Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Michal Piotrowski]

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@gmail.com> wrote:
>>>>
>>>>>>> 831466f4ad2b5fe23dff77edbe6a7c244435e973 is first bad commit
>>>>>>> commit 831466f4ad2b5fe23dff77edbe6a7c244435e973
>>>>>>> Author: Randy Cushman <rcushman_linux@earthlink.net>
>>>>>>> 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@earthlink.net>
>>>>>>>     Signed-off-by: Takashi Iwai <tiwai@suse.de>
>>>>>>>     Signed-off-by: Jaroslav Kysela <perex@suse.cz>
>>>>>>>
>>>>>> 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.

> 
> 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


2.6.21-rc3 witchout 831466f4ad2b5fe23dff77edbe6a7c244435e973

speaker-test -c6

speaker-test 1.0.13

Playback device is default
Stream parameters are 48000Hz, S16_LE, 6 channels
Using 16 octaves of pink noise
Rate set to 48000Hz (requested 48000Hz)
Buffer size range from 2048 to 16384
Period size range from 1024 to 1024
Using max buffer size 16384
Periods = 4
was set period_size = 1024
was set buffer_size = 16384
 0 - Front Left
 4 - Center
 1 - Front Right
 3 - Rear Right
 2 - Rear Left
 5 - LFE

0, 1 works
4, 3, 2, 5 doesn't work

/proc/asound/card0/codec97#0/ac97#0-0

0-0/0: Analog Devices AD1985

PCI Subsys Vendor: 0x1043
PCI Subsys Device: 0x80f3

Revision         : 0x03
Compat. Class    : 0x00
Subsys. Vendor ID: 0xffff
Subsys. ID       : 0xffff

Capabilities     : -headphone out-
DAC resolution   : 20-bit
ADC resolution   : 16-bit
3D enhancement   : No 3D Stereo Enhancement

Current setup
Mic gain         : +20dB [+20dB]
POP path         : pre 3D
Sim. stereo      : off
3D enhancement   : off
Loudness         : off
Mono output      : MIX
Mic select       : Mic1
ADC/DAC loopback : off
Double rate slots: 10/11
Extended ID      : codec=0 rev=0 AMAP LDAC SDAC CDAC DSA=0 SPDIF DRA VRA
Extended status  : SPCV LDAC SDAC CDAC SPDIF=10/11 VRA
PCM front DAC    : 48000Hz
PCM Surr DAC     : 48000Hz
PCM LFE DAC      : 48000Hz
PCM ADC          : 48000Hz
SPDIF Control    : Consumer PCM Category=0x2 Generation=1 Rate=48kHz

                    Gain     Inverted  Buffer delay  Location
Master Out       :   0.0 dBV    -      16/fs         Rear I/O Panel
AUX Out          :   0.0 dBV    -      16/fs         Rear I/O Panel
Center/LFE Out   :   0.0 dBV    -      16/fs         Rear I/O Panel
Mic 1            :   0.0 dBV    -      16/fs         Rear I/O Panel
Mic 2            :   0.0 dBV    -      16/fs         Rear I/O Panel
Line In          :   0.0 dBV    -      16/fs         Rear I/O Panel



AD18XX configuration
Unchained        : 0x1000,0x0000,0x0000
Chained          : 0x0000,0x0000,0x0000


/proc/asound/card0/codec97#0/ac97#0-0+regs

0:00 = 0090
0:02 = 0303
0:04 = 0303
0:06 = 0003
0:08 = 0000
0:0a = 801e
0:0c = 0002
0:0e = 0042
0:10 = 9f9f
0:12 = 0303
0:14 = 0000
0:16 = 0101
0:18 = 0303
0:1a = 0101
0:1c = 0000
0:1e = 0000
0:20 = 0000
0:22 = 0000
0:24 = 0000
0:26 = 000f
0:28 = 03c7
0:2a = 05f1
0:2c = bb80
0:2e = bb80
0:30 = bb80
0:32 = bb80
0:34 = 0000
0:36 = 9f03
0:38 = 0303
0:3a = 2824
0:3c = 0000
0:3e = 0000
0:40 = 0000
0:42 = 0000
0:44 = 0000
0:46 = 0000
0:48 = 0000
0:4a = 0000
0:4c = 0000
0:4e = 0000
0:50 = 0000
0:52 = 0000
0:54 = 0000
0:56 = 0000
0:58 = 0000
0:5a = 0000
0:5c = 0000
0:5e = 0000
0:60 = 8080
0:62 = 0000
0:64 = 0000
0:66 = 0000
0:68 = 0000
0:6a = 0000
0:6c = 0000
0:6e = 0000
0:70 = 0000
0:72 = 0004
0:74 = 1001
0:76 = 64a4
0:78 = 000c
0:7a = 0000
0:7c = 4144
0:7e = 5375

2.6.21-rc3

speaker-test -c6

speaker-test 1.0.13

Playback device is default
Stream parameters are 48000Hz, S16_LE, 6 channels
Using 16 octaves of pink noise
Rate set to 48000Hz (requested 48000Hz)
Buffer size range from 2048 to 16384
Period size range from 1024 to 1024
Using max buffer size 16384
Periods = 4
was set period_size = 1024
was set buffer_size = 16384
 0 - Front Left
 4 - Center
 1 - Front Right
 3 - Rear Right
 2 - Rear Left
 5 - LFE

0, 1 works
4, 3, 2, 5 doesn't work

/proc/asound/card0/codec97#0/ac97#0-0

0-0/0: Analog Devices AD1985

PCI Subsys Vendor: 0x1043
PCI Subsys Device: 0x80f3

Revision         : 0x03
Compat. Class    : 0x00
Subsys. Vendor ID: 0xffff
Subsys. ID       : 0xffff

Capabilities     : -headphone out-
DAC resolution   : 20-bit
ADC resolution   : 16-bit
3D enhancement   : No 3D Stereo Enhancement

Current setup
Mic gain         : +20dB [+20dB]
POP path         : pre 3D
Sim. stereo      : off
3D enhancement   : off
Loudness         : off
Mono output      : MIX
Mic select       : Mic1
ADC/DAC loopback : off
Double rate slots: 10/11
Extended ID      : codec=0 rev=0 AMAP LDAC SDAC CDAC DSA=0 SPDIF DRA VRA
Extended status  : SPCV LDAC SDAC CDAC SPDIF=10/11 VRA
PCM front DAC    : 48000Hz
PCM Surr DAC     : 48000Hz
PCM LFE DAC      : 48000Hz
PCM ADC          : 48000Hz
SPDIF Control    : Consumer PCM Category=0x2 Generation=1 Rate=48kHz

                    Gain     Inverted  Buffer delay  Location
Master Out       :   0.0 dBV    -      16/fs         Rear I/O Panel
AUX Out          :   0.0 dBV    -      16/fs         Rear I/O Panel
Center/LFE Out   :   0.0 dBV    -      16/fs         Rear I/O Panel
Mic 1            :   0.0 dBV    -      16/fs         Rear I/O Panel
Mic 2            :   0.0 dBV    -      16/fs         Rear I/O Panel
Line In          :   0.0 dBV    -      16/fs         Rear I/O Panel



AD18XX configuration
Unchained        : 0x1000,0x0000,0x0000
Chained          : 0x0000,0x0000,0x0000


/proc/asound/card0/codec97#0/ac97#0-0+regs

0:00 = 0090
0:02 = 0303
0:04 = 0303
0:06 = 0003
0:08 = 0000
0:0a = 801e
0:0c = 0002
0:0e = 0042
0:10 = 9f9f
0:12 = 0303
0:14 = 0000
0:16 = 0101
0:18 = 0303
0:1a = 0101
0:1c = 0000
0:1e = 0000
0:20 = 0000
0:22 = 0000
0:24 = 0000
0:26 = 000f
0:28 = 03c7
0:2a = 05f1
0:2c = bb80
0:2e = bb80
0:30 = bb80
0:32 = bb80
0:34 = 0000
0:36 = 9f03
0:38 = 0303
0:3a = 2824
0:3c = 0000
0:3e = 0000
0:40 = 0000
0:42 = 0000
0:44 = 0000
0:46 = 0000
0:48 = 0000
0:4a = 0000
0:4c = 0000
0:4e = 0000
0:50 = 0000
0:52 = 0000
0:54 = 0000
0:56 = 0000
0:58 = 0000
0:5a = 0000
0:5c = 0000
0:5e = 0000
0:60 = 8080
0:62 = 0000
0:64 = 0000
0:66 = 0000
0:68 = 0000
0:6a = 0000
0:6c = 0000
0:6e = 0000
0:70 = 0000
0:72 = 0004
0:74 = 1201
0:76 = 7ca4
0:78 = 000c
0:7a = 0000
0:7c = 4144
0:7e = 5375


> 
> 
> thanks,
> 
> Takashi
> 

Regards,
Michal

-- 
Michal K. K. Piotrowski
LTG - Linux Testers Group (PL)
(http://www.stardust.webpages.pl/ltg/)
LTG - Linux Testers Group (EN)
(http://www.stardust.webpages.pl/linux_testers_group_en/)

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Michal Piotrowski]

On 08/03/07, Michal Piotrowski <michal.k.k.piotrowski@gmail.com> 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@gmail.com> wrote:
> >>>>
> >>>>>>> 831466f4ad2b5fe23dff77edbe6a7c244435e973 is first bad commit
> >>>>>>> commit 831466f4ad2b5fe23dff77edbe6a7c244435e973
> >>>>>>> Author: Randy Cushman <rcushman_linux@earthlink.net>
> >>>>>>> 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@earthlink.net>
> >>>>>>>     Signed-off-by: Takashi Iwai <tiwai@suse.de>
> >>>>>>>     Signed-off-by: Jaroslav Kysela <perex@suse.cz>
> >>>>>>>
> >>>>>> 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.
>
> >
> > 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
>
>
> 2.6.21-rc3 witchout 831466f4ad2b5fe23dff77edbe6a7c244435e973
>
> speaker-test -c6
>
> speaker-test 1.0.13
>
> Playback device is default
> Stream parameters are 48000Hz, S16_LE, 6 channels
> Using 16 octaves of pink noise
> Rate set to 48000Hz (requested 48000Hz)
> Buffer size range from 2048 to 16384
> Period size range from 1024 to 1024
> Using max buffer size 16384
> Periods = 4
> was set period_size = 1024
> was set buffer_size = 16384
>  0 - Front Left
>  4 - Center
>  1 - Front Right
>  3 - Rear Right
>  2 - Rear Left
>  5 - LFE
>
> 0, 1 works
> 4, 3, 2, 5 doesn't work
>
> /proc/asound/card0/codec97#0/ac97#0-0
>
> 0-0/0: Analog Devices AD1985
>
> PCI Subsys Vendor: 0x1043
> PCI Subsys Device: 0x80f3
>
> Revision         : 0x03
> Compat. Class    : 0x00
> Subsys. Vendor ID: 0xffff
> Subsys. ID       : 0xffff
>
> Capabilities     : -headphone out-
> DAC resolution   : 20-bit
> ADC resolution   : 16-bit
> 3D enhancement   : No 3D Stereo Enhancement
>
> Current setup
> Mic gain         : +20dB [+20dB]
> POP path         : pre 3D
> Sim. stereo      : off
> 3D enhancement   : off
> Loudness         : off
> Mono output      : MIX
> Mic select       : Mic1
> ADC/DAC loopback : off
> Double rate slots: 10/11
> Extended ID      : codec=0 rev=0 AMAP LDAC SDAC CDAC DSA=0 SPDIF DRA VRA
> Extended status  : SPCV LDAC SDAC CDAC SPDIF=10/11 VRA
> PCM front DAC    : 48000Hz
> PCM Surr DAC     : 48000Hz
> PCM LFE DAC      : 48000Hz
> PCM ADC          : 48000Hz
> SPDIF Control    : Consumer PCM Category=0x2 Generation=1 Rate=48kHz
>
>                     Gain     Inverted  Buffer delay  Location
> Master Out       :   0.0 dBV    -      16/fs         Rear I/O Panel
> AUX Out          :   0.0 dBV    -      16/fs         Rear I/O Panel
> Center/LFE Out   :   0.0 dBV    -      16/fs         Rear I/O Panel
> Mic 1            :   0.0 dBV    -      16/fs         Rear I/O Panel
> Mic 2            :   0.0 dBV    -      16/fs         Rear I/O Panel
> Line In          :   0.0 dBV    -      16/fs         Rear I/O Panel
>
>
>
> AD18XX configuration
> Unchained        : 0x1000,0x0000,0x0000
> Chained          : 0x0000,0x0000,0x0000
>
>
> /proc/asound/card0/codec97#0/ac97#0-0+regs
>
> 0:00 = 0090
> 0:02 = 0303
> 0:04 = 0303
> 0:06 = 0003
> 0:08 = 0000
> 0:0a = 801e
> 0:0c = 0002
> 0:0e = 0042
> 0:10 = 9f9f
> 0:12 = 0303
> 0:14 = 0000
> 0:16 = 0101
> 0:18 = 0303
> 0:1a = 0101
> 0:1c = 0000
> 0:1e = 0000
> 0:20 = 0000
> 0:22 = 0000
> 0:24 = 0000
> 0:26 = 000f
> 0:28 = 03c7
> 0:2a = 05f1
> 0:2c = bb80
> 0:2e = bb80
> 0:30 = bb80
> 0:32 = bb80
> 0:34 = 0000
> 0:36 = 9f03
> 0:38 = 0303
> 0:3a = 2824
> 0:3c = 0000
> 0:3e = 0000
> 0:40 = 0000
> 0:42 = 0000
> 0:44 = 0000
> 0:46 = 0000
> 0:48 = 0000
> 0:4a = 0000
> 0:4c = 0000
> 0:4e = 0000
> 0:50 = 0000
> 0:52 = 0000
> 0:54 = 0000
> 0:56 = 0000
> 0:58 = 0000
> 0:5a = 0000
> 0:5c = 0000
> 0:5e = 0000
> 0:60 = 8080
> 0:62 = 0000
> 0:64 = 0000
> 0:66 = 0000
> 0:68 = 0000
> 0:6a = 0000
> 0:6c = 0000
> 0:6e = 0000
> 0:70 = 0000
> 0:72 = 0004
> 0:74 = 1001
> 0:76 = 64a4
> 0:78 = 000c
> 0:7a = 0000
> 0:7c = 4144
> 0:7e = 5375
>
> 2.6.21-rc3
>
> speaker-test -c6
>
> speaker-test 1.0.13
>
> Playback device is default
> Stream parameters are 48000Hz, S16_LE, 6 channels
> Using 16 octaves of pink noise
> Rate set to 48000Hz (requested 48000Hz)
> Buffer size range from 2048 to 16384
> Period size range from 1024 to 1024
> Using max buffer size 16384
> Periods = 4
> was set period_size = 1024
> was set buffer_size = 16384
>  0 - Front Left
>  4 - Center
>  1 - Front Right
>  3 - Rear Right
>  2 - Rear Left
>  5 - LFE
>
> 0, 1 works
> 4, 3, 2, 5 doesn't work
>
> /proc/asound/card0/codec97#0/ac97#0-0
>
> 0-0/0: Analog Devices AD1985
>
> PCI Subsys Vendor: 0x1043
> PCI Subsys Device: 0x80f3
>
> Revision         : 0x03
> Compat. Class    : 0x00
> Subsys. Vendor ID: 0xffff
> Subsys. ID       : 0xffff
>
> Capabilities     : -headphone out-
> DAC resolution   : 20-bit
> ADC resolution   : 16-bit
> 3D enhancement   : No 3D Stereo Enhancement
>
> Current setup
> Mic gain         : +20dB [+20dB]
> POP path         : pre 3D
> Sim. stereo      : off
> 3D enhancement   : off
> Loudness         : off
> Mono output      : MIX
> Mic select       : Mic1
> ADC/DAC loopback : off
> Double rate slots: 10/11
> Extended ID      : codec=0 rev=0 AMAP LDAC SDAC CDAC DSA=0 SPDIF DRA VRA
> Extended status  : SPCV LDAC SDAC CDAC SPDIF=10/11 VRA
> PCM front DAC    : 48000Hz
> PCM Surr DAC     : 48000Hz
> PCM LFE DAC      : 48000Hz
> PCM ADC          : 48000Hz
> SPDIF Control    : Consumer PCM Category=0x2 Generation=1 Rate=48kHz
>
>                     Gain     Inverted  Buffer delay  Location
> Master Out       :   0.0 dBV    -      16/fs         Rear I/O Panel
> AUX Out          :   0.0 dBV    -      16/fs         Rear I/O Panel
> Center/LFE Out   :   0.0 dBV    -      16/fs         Rear I/O Panel
> Mic 1            :   0.0 dBV    -      16/fs         Rear I/O Panel
> Mic 2            :   0.0 dBV    -      16/fs         Rear I/O Panel
> Line In          :   0.0 dBV    -      16/fs         Rear I/O Panel
>
>
>
> AD18XX configuration
> Unchained        : 0x1000,0x0000,0x0000
> Chained          : 0x0000,0x0000,0x0000
>
>
> /proc/asound/card0/codec97#0/ac97#0-0+regs
>
> 0:00 = 0090
> 0:02 = 0303
> 0:04 = 0303
> 0:06 = 0003
> 0:08 = 0000
> 0:0a = 801e
> 0:0c = 0002
> 0:0e = 0042
> 0:10 = 9f9f
> 0:12 = 0303
> 0:14 = 0000
> 0:16 = 0101
> 0:18 = 0303
> 0:1a = 0101
> 0:1c = 0000
> 0:1e = 0000
> 0:20 = 0000
> 0:22 = 0000
> 0:24 = 0000
> 0:26 = 000f
> 0:28 = 03c7
> 0:2a = 05f1
> 0:2c = bb80
> 0:2e = bb80
> 0:30 = bb80
> 0:32 = bb80
> 0:34 = 0000
> 0:36 = 9f03
> 0:38 = 0303
> 0:3a = 2824
> 0:3c = 0000
> 0:3e = 0000
> 0:40 = 0000
> 0:42 = 0000
> 0:44 = 0000
> 0:46 = 0000
> 0:48 = 0000
> 0:4a = 0000
> 0:4c = 0000
> 0:4e = 0000
> 0:50 = 0000
> 0:52 = 0000
> 0:54 = 0000
> 0:56 = 0000
> 0:58 = 0000
> 0:5a = 0000
> 0:5c = 0000
> 0:5e = 0000
> 0:60 = 8080
> 0:62 = 0000
> 0:64 = 0000
> 0:66 = 0000
> 0:68 = 0000
> 0:6a = 0000
> 0:6c = 0000
> 0:6e = 0000
> 0:70 = 0000
> 0:72 = 0004
> 0:74 = 1201
> 0:76 = 7ca4
> 0:78 = 000c
> 0:7a = 0000
> 0:7c = 4144
> 0:7e = 5375
>
>
> >
> >
> > thanks,
> >
> > Takashi
> >
>

s/witchout/without
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Takashi Iwai]

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@gmail.com> wrote:
> >>>>
> >>>>>>> 831466f4ad2b5fe23dff77edbe6a7c244435e973 is first bad commit
> >>>>>>> commit 831466f4ad2b5fe23dff77edbe6a7c244435e973
> >>>>>>> Author: Randy Cushman <rcushman_linux@earthlink.net>
> >>>>>>> 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@earthlink.net>
> >>>>>>>     Signed-off-by: Takashi Iwai <tiwai@suse.de>
> >>>>>>>     Signed-off-by: Jaroslav Kysela <perex@suse.cz>
> >>>>>>>
> >>>>>> 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

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Michal Piotrowski]

On 08/03/07, Takashi Iwai <tiwai@suse.de> wrote:
> At Thu, 08 Mar 2007 16:42:26 +0100,
> Michal Piotrowski wrote:
> >
> > Takashi Iwai napisał(a):
> > > 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.

It still doesn't work (at least on clean 2.6.21-rc3).

Regards,
Michal

-- 
Michal K. K. Piotrowski
LTG - Linux Testers Group (PL)
(http://www.stardust.webpages.pl/ltg/)
LTG - Linux Testers Group (EN)
(http://www.stardust.webpages.pl/linux_testers_group_en/)
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Takashi Iwai]

At Thu, 8 Mar 2007 17:26:48 +0100,
Michal Piotrowski wrote:
> 
> On 08/03/07, Takashi Iwai <tiwai@suse.de> wrote:
> > At Thu, 08 Mar 2007 16:42:26 +0100,
> > Michal Piotrowski wrote:
> > >
> > > Takashi Iwai napisał(a):
> > > > 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.
> 
> It still doesn't work (at least on clean 2.6.21-rc3).

That's expected.  But it should work with the fixed kernel.
Let me know if it still doesn't work with the patched kernel.


thanks,

Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Takashi Iwai]

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

Just to make sure: it means that a2.txt is a working one and the
anoter is non-working, right?


Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Michal Piotrowski]

On 08/03/07, Takashi Iwai <tiwai@suse.de> 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
>
> Just to make sure: it means that a2.txt is a working one and the
> anoter is non-working, right?

Right.

>
>
> Takashi
>

Regards,
Michal

-- 
Michal K. K. Piotrowski
LTG - Linux Testers Group (PL)
(http://www.stardust.webpages.pl/ltg/)
LTG - Linux Testers Group (EN)
(http://www.stardust.webpages.pl/linux_testers_group_en/)

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Takashi Iwai]

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?


Takashi

diff -r 6888d4cd251d sound/pci/ac97/ac97_patch.c
--- a/sound/pci/ac97/ac97_patch.c	Thu Mar 08 12:53:55 2007 +0100
+++ b/sound/pci/ac97/ac97_patch.c	Thu Mar 08 17:10:52 2007 +0100
@@ -1950,9 +1950,9 @@ static void ad1888_update_jacks(struct s
 static void ad1888_update_jacks(struct snd_ac97 *ac97)
 {
 	unsigned short val = 0;
-	if (! is_shared_linein(ac97))
+	if (is_shared_linein(ac97))
 		val |= (1 << 12);
-	if (! is_shared_micin(ac97))
+	if (is_shared_micin(ac97))
 		val |= (1 << 11);
 	/* shared Line-In */
 	snd_ac97_update_bits(ac97, AC97_AD_MISC, (1 << 11) | (1 << 12), val);

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Michal Piotrowski]

On 08/03/07, Takashi Iwai <tiwai@suse.de> 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)

Regards,
Michal

-- 
Michal K. K. Piotrowski
LTG - Linux Testers Group (PL)
(http://www.stardust.webpages.pl/ltg/)
LTG - Linux Testers Group (EN)
(http://www.stardust.webpages.pl/linux_testers_group_en/)

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Takashi Iwai]

At Thu, 8 Mar 2007 17:52:43 +0100,
Michal Piotrowski wrote:
> 
> On 08/03/07, Takashi Iwai <tiwai@suse.de> 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

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Michal Piotrowski]

On 08/03/07, Takashi Iwai <tiwai@suse.de> wrote:
> At Thu, 8 Mar 2007 17:52:43 +0100,
> Michal Piotrowski wrote:
> >
> > On 08/03/07, Takashi Iwai <tiwai@suse.de> 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?

Now I've got:
rear left as center
rear right as LFE

:)

>
>
> Takashi
>

Regards,
Michal

-- 
Michal K. K. Piotrowski
LTG - Linux Testers Group (PL)
(http://www.stardust.webpages.pl/ltg/)
LTG - Linux Testers Group (EN)
(http://www.stardust.webpages.pl/linux_testers_group_en/)

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Randy Cushman]

Takashi Iwai wrote:
> At Thu, 8 Mar 2007 17:52:43 +0100,
> Michal Piotrowski wrote:
>   
>> On 08/03/07, Takashi Iwai <tiwai@suse.de> 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).

Please double-check your mixer settings.  I added a byte to a structure, 
so the mixer settings can get messed up switching between alsa-kernel 
versions.

I don't understand why the other bit (OMS, 0x74 bit D9) is set.  This 
may have been an oversight on my part.

I am modifying alsa-kernel to run as an AD1985 on my system, to try to 
check these bits.

Randy Cushman



-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Takashi Iwai]

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 <tiwai@suse.de> 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

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Randy Cushman]

Takashi Iwai wrote:
> 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 <tiwai@suse.de> 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
>
>
>   
Takashi,

I need to think this through before I can confirm.  Something doesn't 
seem right.

Let me figure out why I wrote ad1986_update_jacks() to be so different 
from ad1888_update_jacks().

I'll keep you posted.

Randy Cushman



-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

Patch: Shared jack control logic for AD1985 and others

[Randy Cushman]

[-- Attachment #1: Type: text/plain, Size: 309 bytes --]

Takashi and Michal,

Here is a patch that I believe will correct the issue Michal identified.

Takashi's patch was close, but he overlooked that I also had the logic 
for the OMS bit reversed.  For the AD1985 Surround would have worked 
correctly, but Center/LFE still would have been broken.

Randy Cushman


[-- Attachment #2: ac97_ad_shared.patch --]
[-- Type: text/plain, Size: 1797 bytes --]

Summary: fix AD shared shared jack control logic

This patch fixes the control logic for shared
Microphone in/Center+LFE out and Line in/Surround out jacks
for Analog Devices AD1888, AD1980 and AD1985 AC '97 CODECs.

Signed-off-by: Randy Cushman <rcushman_linux@earthlink.net>


diff -r e4f79cb873e5 pci/ac97/ac97_patch.c
--- a/pci/ac97/ac97_patch.c	Tue Mar 06 11:29:05 2007 +0100
+++ b/pci/ac97/ac97_patch.c	Thu Mar 08 17:08:52 2007 -0500
@@ -1950,9 +1950,11 @@ static void ad1888_update_jacks(struct s
 static void ad1888_update_jacks(struct snd_ac97 *ac97)
 {
 	unsigned short val = 0;
-	if (! is_shared_linein(ac97))
+	/* clear LODIS if shared jack is to be used for Surround out */
+	if (is_shared_linein(ac97))
 		val |= (1 << 12);
-	if (! is_shared_micin(ac97))
+	/* clear CLDIS if shared jack is to be used for C/LFE out */
+	if (is_shared_micin(ac97))
 		val |= (1 << 11);
 	/* shared Line-In */
 	snd_ac97_update_bits(ac97, AC97_AD_MISC, (1 << 11) | (1 << 12), val);
@@ -2124,8 +2126,9 @@ static void ad1985_update_jacks(struct s
 static void ad1985_update_jacks(struct snd_ac97 *ac97)
 {
 	ad1888_update_jacks(ac97);
+	/* clear OMS if shared jack is to be used for C/LFE out */
 	snd_ac97_update_bits(ac97, AC97_AD_SERIAL_CFG, 1 << 9,
-			     is_shared_micin(ac97) ? 0 : 1 << 9);
+			     is_shared_micin(ac97) ? 1 << 9 : 0);
 }
 
 static int patch_ad1985_specific(struct snd_ac97 *ac97)
@@ -2406,9 +2409,9 @@ static void ad1986_update_jacks(struct s
 	unsigned short ser_val;
 
 	/* disable SURROUND and CENTER/LFE if not surround mode */
-	if (! is_surround_on(ac97))
+	if (!is_surround_on(ac97))
 		misc_val |= AC97_AD1986_SODIS;
-	if (! is_clfe_on(ac97))
+	if (!is_clfe_on(ac97))
 		misc_val |= AC97_AD1986_CLDIS;
 
 	/* select line input (default=LINE_IN, SURROUND or MIC_1/2) */

[-- Attachment #3: Type: text/plain, Size: 345 bytes --]

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

[-- Attachment #4: Type: text/plain, Size: 161 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/alsa-devel

Re: Patch: Shared jack control logic for AD1985 and others

[Takashi Iwai]

At Thu, 08 Mar 2007 18:05:47 -0500,
Randy Cushman wrote:
> 
> Takashi and Michal,
> 
> Here is a patch that I believe will correct the issue Michal identified.
> 
> Takashi's patch was close, but he overlooked that I also had the logic 
> for the OMS bit reversed.  For the AD1985 Surround would have worked 
> correctly, but Center/LFE still would have been broken.
> 
> Randy Cushman

Thanks, applied to ALSA tree now.


Takashi

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

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

[Michal Piotrowski]

On 07/03/07, Andrew Morton <akpm@linux-foundation.org> wrote:
> On Wed, 7 Mar 2007 20:41:30 +0100 "Michal Piotrowski" <michal.k.k.piotrowski@gmail.com> wrote:
>
> > > > 831466f4ad2b5fe23dff77edbe6a7c244435e973 is first bad commit
> > > > commit 831466f4ad2b5fe23dff77edbe6a7c244435e973
> > > > Author: Randy Cushman <rcushman_linux@earthlink.net>
> > > > 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@earthlink.net>
> > > >     Signed-off-by: Takashi Iwai <tiwai@suse.de>
> > > >     Signed-off-by: Jaroslav Kysela <perex@suse.cz>
> > > >
> > >
> > > Is 2.6.21-rc3 similarly broken?
> >
> > Yes.
> >
>
> OK.  Can you please confirm that applying the below fixes 2.6.21-rc3?

Yes, I can confirm that.

>
> If so, I'll queue it up for application in a week or two if there is no
> progress on this.
>

Regards,
Michal

-- 
Michal K. K. Piotrowski
LTG - Linux Testers Group (PL)
(http://www.stardust.webpages.pl/ltg/)
LTG - Linux Testers Group (EN)
(http://www.stardust.webpages.pl/linux_testers_group_en/)

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV