Re: Re: [alsa-cvslog] CVS: alsa-kernel/pci/emu10k1 emu10k1_main.c,1.12,1.13

[Takashi Iwai <tiwai@suse.de>]

At Tue, 16 Apr 2002 09:15:10 +0200 (CEST),
Jaroslav wrote:
> 
> On Mon, 15 Apr 2002, Markus Plail wrote:
> 
> > Hi Takashi!
> > 
> > * Takashi Iwai writes:
> > > At Thu, 11 Apr 2002 15:42:23 +0100,
> > > James Courtier-Dutton wrote:
> > >> 
> > >> [1  <text/plain; us-ascii (7bit)>]
> > >> Takashi Iwai wrote:
> > >> 
> > >> >Update of /cvsroot/alsa/alsa-kernel/pci/emu10k1
> > >> >In directory usw-pr-cvs1:/tmp/cvs-serv9192
> > >> >
> > >> >Modified Files:
> > >> >	emu10k1_main.c 
> > >> >Log Message:
> > >> >fixed default mask values
> > >> >(these change won't affect the actual behavior, though)
> > 
> > [snip]
> > 
> > >> This seems to have fixed AC3 Passthru mode on SB Live (rev 7) cards.
> > >> Just so you know, all SB Live cards worked before for AC3 Passthru 
> > >> except the rev 7 cards.
> > >> I just got a report on the xine-user list about it.
> > >> Ok, so you did not know you were fixing it, but well done anyway!!!! ;-)
> > > err, frankly speaking, i still don't figure out why with this change
> > > the pass-through works.  i'm glad to to hear a good news, anyway ;)
> > 
> > Just wanted to let you know that it wasn't the changes in emu10k1_main.c,
> > but the changes in emumixer.c that made my (I was the one reporting it on
> > the xine ML) rev07 card working.
> 
> The joke is, that this behaviour is caused by a mistake. Here is the right 
> patch against current CVS:

wow :)


> At least, we know that for rev07 cards we shouldn't modify GPOUT0. These 
> cards have separated the digital output or shared with an analog output?

Markus, could you try the latest cvs version, which fixes the "real"
behavior, whether digital out works?
if it doesn't work again, then definitely GPOUT0 is the spot.

looking at OSS source codes, i found GPOUT0 is necessary only for
5.1 cards (and seems for Audigy too - i need to fix again).

so i'm not sure whether we can determine it from revision number.


Takashi