Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 24 Aug 2003 11:36:49 -0700
From:      Orion Hodson <orion@FreeBSD.org>
To:        obrien@FreeBSD.org
Cc:        cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/dev/sound/pcm ac97.c ac97.h 
Message-ID:  <200308241836.h7OIan91084617@puma.icir.org>
In-Reply-To: Your message of "Sun, 24 Aug 2003 10:28:55 PDT." <20030824172855.GA98809@dragon.nuxi.com> 

next in thread | previous in thread | raw e-mail | index | archive | help
/-- "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




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200308241836.h7OIan91084617>