Re: Re: [alsa-cvslog] CVS: BEFORE REVERT [SUCCESS] C-Media CMI9761 -A- type, WORKING MASTER, PCM ON 1. DXS SLIDER!

[Takashi Iwai <tiwai@suse.de>]

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

At Wed, 16 Feb 2005 22:29:40 +0100,
thomas schorpp wrote:
> 
> - arts works with pcm fine
> - working master vol slider(!) (kmix)
> - working pcm vol regulation on 1. dxs slider (rest 3 no sound reaction)

(snip)

> > Revert the last addition for CM9761A support.  The codec doesn't
> > support the real control of Master
> 
> yes, it does fine for my chip , sorry.
> 
> /PCM volumes, too.

Hmm, I'm really confused.  Both the comments above look
contradictionary.

Let me ask again:  In the older version, does PCM volume work, that
is, it really changes the output volume (not only mute/unmute)?
Does Master volume work as well on your chip?


Note that the Master and PCM volumes are _removed_ because they don't
work and are only confusing.  It's intentional.  They _must_ disappear
in the latest version.

If CM9761-A- provides the real PCM/Master volume control, we should
get them back again.  But, this essential information has been
missing.   Please let me know _this_ clearly above all subtle things!


> > Instead, fixed the default multi-channel register setting now.
> > 
> 
> why?
> not verified. source of info? are you sure thats the -A- type, you got 
> for testing? who verified?

Very much confision of debug information...


> oops! sound gone after doing this:

How about the attached patch?


Takashi

[-- Attachment #2: Type: text/plain, Size: 821 bytes --]

Index: alsa-kernel/pci/ac97/ac97_patch.c
===================================================================
RCS file: /suse/tiwai/cvs/alsa/alsa-kernel/pci/ac97/ac97_patch.c,v
retrieving revision 1.76
diff -u -r1.76 ac97_patch.c
--- alsa-kernel/pci/ac97/ac97_patch.c	16 Feb 2005 18:25:30 -0000	1.76
+++ alsa-kernel/pci/ac97/ac97_patch.c	17 Feb 2005 11:11:01 -0000
@@ -2038,8 +2038,8 @@
 
 	/* CM9761 has no Master and PCM volume although the register reacts */
 	ac97->flags |= AC97_HAS_NO_MASTER_VOL | AC97_HAS_NO_PCM_VOL;
-	snd_ac97_write_cache(ac97, AC97_MASTER, 0x8000);
-	snd_ac97_write_cache(ac97, AC97_PCM, 0x8000);
+	snd_ac97_write_cache(ac97, AC97_MASTER, 0x8808);
+	snd_ac97_write_cache(ac97, AC97_PCM, 0x8808);
 
 	ac97->spec.dev_flags = 0; /* 1 = model 82 revision B */
 	if (ac97->id == AC97_ID_CM9761_82) {