Skip to contentSkip to conversation

ALSA: cs5535audio: stick AD1888 bitshift values into a header file

PATCH Applied Andres Salomon 2 participants 2 messages Nov 5, 2008 – Nov 6, 2008 Sound

lore
Share & tools
lore

Andres Salomon

author
#0 lore raw

PATCH 1/4 ALSA: cs5535audio: stick AD1888 bitshift values into a header file

We'd like to use the High Pass Filter and V_REFOUT bitshift values elsewhere, so stick them into a ac97_codec.h.

Signed-off-by Andres Salomon <[email protected]>
---
 include/sound/ac97_codec.h  |    2 ++
 sound/pci/ac97/ac97_patch.c |    5 +++--
 2 files changed, 5 insertions(+), 2 deletions(-)
include/sound/ac97_codec.h +2 -0
index 9c309da..251fc1c 100644
@@ -281,10 +281,12 @@
 /* specific - Analog Devices */
 #define AC97_AD_TEST		0x5a	/* test register */
 #define AC97_AD_TEST2		0x5c	/* undocumented test register 2 */
+#define AC97_AD_HPFD_SHIFT	12	/* High Pass Filter Disable */
 #define AC97_AD_CODEC_CFG	0x70	/* codec configuration */
 #define AC97_AD_JACK_SPDIF	0x72	/* Jack Sense & S/PDIF */
 #define AC97_AD_SERIAL_CFG	0x74	/* Serial Configuration */
 #define AC97_AD_MISC		0x76	/* Misc Control Bits */
+#define AC97_AD_VREFD_SHIFT	2	/* V_REFOUT Disable (AD1888) */
 
 /* specific - Cirrus Logic */
 #define AC97_CSR_ACMODE		0x5e	/* AC Mode Register */
sound/pci/ac97/ac97_patch.c +3 -2
index 6e831af..7ad25f4 100644
@@ -2054,8 +2054,9 @@ static const struct snd_kcontrol_new snd_ac97_ad1888_controls[] = {
 		.get = snd_ac97_ad1888_lohpsel_get,
 		.put = snd_ac97_ad1888_lohpsel_put
 	},
-	AC97_SINGLE("V_REFOUT Enable", AC97_AD_MISC, 2, 1, 1),
-	AC97_SINGLE("High Pass Filter Enable", AC97_AD_TEST2, 12, 1, 1),
+	AC97_SINGLE("V_REFOUT Enable", AC97_AD_MISC, AC97_AD_VREFD_SHIFT, 1, 1),
+	AC97_SINGLE("High Pass Filter Enable", AC97_AD_TEST2,
+			AC97_AD_HPFD_SHIFT, 1, 1),
 	AC97_SINGLE("Spread Front to Surround and Center/LFE", AC97_AD_MISC, 7, 1, 0),
 	{
 		.iface = SNDRV_CTL_ELEM_IFACE_MIXER,
Signature · 1 line
1.5.6.5
Applied
#1 lore raw

At Wed, 5 Nov 2008 17:29:53 -0500,

Quoted 6 lines · Andres Salomon

We'd like to use the High Pass Filter and V_REFOUT bitshift values elsewhere, so stick them into a ac97_codec.h.

Signed-off-by Andres Salomon <[email protected]>

Thanks, applied patches from 1 to 3 now as they are trivial fixes.

The patch 4 is pending. As Jaya comments, please split the patch to his original one and additional fixes.

thanks,

Takashi