cvs commit: src/sys/dev/sound/pcm ac97.c ac97.h

[Orion Hodson]

/-- "David O'Brien" wrote:
| On Thu, Aug 21, 2003 at 08:50:33AM -0700, Orion Hodson wrote:
| > orion       2003/08/21 08:50:33 PDT
| >   Modified files:
| >     sys/dev/sound/pcm    ac97.c ac97.h 
| >   Log:
| >   Apply Rudolf Cejka's patch for:
| >   o AD1980 hook.
| >   o ac97_fix_auxout.
| >   and:
| >   o Associate AC97_MIX_AUXOUT with SOUND_MIXER_OGAIN rather than
| >   SOUND_MIXER_MONITOR.
| >   o Add ac97_fix_tone to remove tone controls from mixer if invalid.
| >   
| >   Revision  Changes    Path
| >   1.43      +42 -47    src/sys/dev/sound/pcm/ac97.c
| > http://cvsweb.freebsd.org/src/sys/dev/sound/pcm/ac97.c.diff?r1=1.42&r2=1.43
| >   1.14      +1 -1      src/sys/dev/sound/pcm/ac97.h
| > http://cvsweb.freebsd.org/src/sys/dev/sound/pcm/ac97.h.diff?r1=1.13&r2=1.14
| 
|     -#define AC97_MUTE      0x8000
|     +#define AC97_MUTE      0x8080
| 
| this now conficts with gnu/dev/sound/pci/emu10k1-ac97.h which is needed
| for Audigy2 support.  Why the change?

Because this is the appropriate value for the center/LFE and surround volume 
controls (ac97 registers 0x36 and 0x38).  The latter is used in Rudolf's patch.

For the registers where 0x8000 corresponds to the mute bit, it's not-critical 
whether the additional bit is set or not: the bit is marked as reserved in the 
spec and by the specs words may be written, but must return 0 upon read back.

What is the status of the Audigy2 support?
- Orion