Commit 45d76056 authored by Daniel Mack's avatar Daniel Mack Committed by Takashi Iwai

ALSA: usb-mixer: use defines from audio.h

No need for the private enum.
Signed-off-by: default avatarDaniel Mack <daniel@caiaq.de>
Cc: Clemens Ladisch <clemens@ladisch.de>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 7e847894
...@@ -218,7 +218,8 @@ static int parse_audio_format_rates_v2(struct snd_usb_audio *chip, ...@@ -218,7 +218,8 @@ static int parse_audio_format_rates_v2(struct snd_usb_audio *chip,
/* get the number of sample rates first by only fetching 2 bytes */ /* get the number of sample rates first by only fetching 2 bytes */
ret = snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0), UAC2_CS_RANGE, ret = snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0), UAC2_CS_RANGE,
USB_TYPE_CLASS | USB_RECIP_INTERFACE | USB_DIR_IN, USB_TYPE_CLASS | USB_RECIP_INTERFACE | USB_DIR_IN,
0x0100, chip->clock_id << 8, tmp, sizeof(tmp), 1000); UAC2_CS_CONTROL_SAM_FREQ << 8, chip->clock_id << 8,
tmp, sizeof(tmp), 1000);
if (ret < 0) { if (ret < 0) {
snd_printk(KERN_ERR "unable to retrieve number of sample rates\n"); snd_printk(KERN_ERR "unable to retrieve number of sample rates\n");
...@@ -236,7 +237,8 @@ static int parse_audio_format_rates_v2(struct snd_usb_audio *chip, ...@@ -236,7 +237,8 @@ static int parse_audio_format_rates_v2(struct snd_usb_audio *chip,
/* now get the full information */ /* now get the full information */
ret = snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0), UAC2_CS_RANGE, ret = snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0), UAC2_CS_RANGE,
USB_TYPE_CLASS | USB_RECIP_INTERFACE | USB_DIR_IN, USB_TYPE_CLASS | USB_RECIP_INTERFACE | USB_DIR_IN,
0x0100, chip->clock_id << 8, data, data_size, 1000); UAC2_CS_CONTROL_SAM_FREQ << 8, chip->clock_id << 8,
data, data_size, 1000);
if (ret < 0) { if (ret < 0) {
snd_printk(KERN_ERR "unable to retrieve sample rate range\n"); snd_printk(KERN_ERR "unable to retrieve sample rate range\n");
......
...@@ -136,21 +136,6 @@ struct usb_mixer_elem_info { ...@@ -136,21 +136,6 @@ struct usb_mixer_elem_info {
u8 initialized; u8 initialized;
}; };
enum {
USB_FEATURE_NONE = 0,
USB_FEATURE_MUTE = 1,
USB_FEATURE_VOLUME,
USB_FEATURE_BASS,
USB_FEATURE_MID,
USB_FEATURE_TREBLE,
USB_FEATURE_GEQ,
USB_FEATURE_AGC,
USB_FEATURE_DELAY,
USB_FEATURE_BASSBOOST,
USB_FEATURE_LOUDNESS
};
enum { enum {
USB_MIXER_BOOLEAN, USB_MIXER_BOOLEAN,
USB_MIXER_INV_BOOLEAN, USB_MIXER_INV_BOOLEAN,
...@@ -954,7 +939,7 @@ static void build_feature_ctl(struct mixer_build *state, unsigned char *desc, ...@@ -954,7 +939,7 @@ static void build_feature_ctl(struct mixer_build *state, unsigned char *desc,
control++; /* change from zero-based to 1-based value */ control++; /* change from zero-based to 1-based value */
if (control == USB_FEATURE_GEQ) { if (control == UAC_GRAPHIC_EQUALIZER_CONTROL) {
/* FIXME: not supported yet */ /* FIXME: not supported yet */
return; return;
} }
...@@ -1001,8 +986,8 @@ static void build_feature_ctl(struct mixer_build *state, unsigned char *desc, ...@@ -1001,8 +986,8 @@ static void build_feature_ctl(struct mixer_build *state, unsigned char *desc,
kctl->id.name, sizeof(kctl->id.name)); kctl->id.name, sizeof(kctl->id.name));
switch (control) { switch (control) {
case USB_FEATURE_MUTE: case UAC_MUTE_CONTROL:
case USB_FEATURE_VOLUME: case UAC_VOLUME_CONTROL:
/* determine the control name. the rule is: /* determine the control name. the rule is:
* - if a name id is given in descriptor, use it. * - if a name id is given in descriptor, use it.
* - if the connected input can be determined, then use the name * - if the connected input can be determined, then use the name
...@@ -1029,9 +1014,9 @@ static void build_feature_ctl(struct mixer_build *state, unsigned char *desc, ...@@ -1029,9 +1014,9 @@ static void build_feature_ctl(struct mixer_build *state, unsigned char *desc,
len = append_ctl_name(kctl, " Playback"); len = append_ctl_name(kctl, " Playback");
} }
} }
append_ctl_name(kctl, control == USB_FEATURE_MUTE ? append_ctl_name(kctl, control == UAC_MUTE_CONTROL ?
" Switch" : " Volume"); " Switch" : " Volume");
if (control == USB_FEATURE_VOLUME) { if (control == UAC_VOLUME_CONTROL) {
kctl->tlv.c = mixer_vol_tlv; kctl->tlv.c = mixer_vol_tlv;
kctl->vd[0].access |= kctl->vd[0].access |=
SNDRV_CTL_ELEM_ACCESS_TLV_READ | SNDRV_CTL_ELEM_ACCESS_TLV_READ |
...@@ -1120,7 +1105,7 @@ static int parse_audio_feature_unit(struct mixer_build *state, int unitid, void ...@@ -1120,7 +1105,7 @@ static int parse_audio_feature_unit(struct mixer_build *state, int unitid, void
snd_printk(KERN_INFO snd_printk(KERN_INFO
"usbmixer: master volume quirk for PCM2702 chip\n"); "usbmixer: master volume quirk for PCM2702 chip\n");
/* disable non-functional volume control */ /* disable non-functional volume control */
master_bits &= ~(1 << (USB_FEATURE_VOLUME - 1)); master_bits &= ~UAC_FU_VOLUME;
break; break;
} }
if (channels > 0) if (channels > 0)
......
...@@ -85,8 +85,8 @@ static struct usbmix_name_map extigy_map[] = { ...@@ -85,8 +85,8 @@ static struct usbmix_name_map extigy_map[] = {
/* 16: MU (w/o controls) */ /* 16: MU (w/o controls) */
{ 17, NULL, 1 }, /* DISABLED: PU-switch (any effect?) */ { 17, NULL, 1 }, /* DISABLED: PU-switch (any effect?) */
{ 17, "Channel Routing", 2 }, /* PU: mode select */ { 17, "Channel Routing", 2 }, /* PU: mode select */
{ 18, "Tone Control - Bass", USB_FEATURE_BASS }, /* FU */ { 18, "Tone Control - Bass", UAC_BASS_CONTROL }, /* FU */
{ 18, "Tone Control - Treble", USB_FEATURE_TREBLE }, /* FU */ { 18, "Tone Control - Treble", UAC_TREBLE_CONTROL }, /* FU */
{ 18, "Master Playback" }, /* FU; others */ { 18, "Master Playback" }, /* FU; others */
/* 19: OT speaker */ /* 19: OT speaker */
/* 20: OT headphone */ /* 20: OT headphone */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment