aboutsummaryrefslogtreecommitdiffstats
path: root/sound/usb
diff options
context:
space:
mode:
authorEldad Zack <eldad@fogrefinery.com>2013-10-06 16:31:13 -0400
committerTakashi Iwai <tiwai@suse.de>2013-10-07 05:22:03 -0400
commitdf23a2466a4961092625b487ead60f526e401062 (patch)
tree3da74828efa301829c66b6b13397e14df572ae16 /sound/usb
parent06613f547a4f62c2201f70b24807d9edd4d733b0 (diff)
ALSA: usb-audio: rename alt_idx to altsetting
As Clemens Ladisch kindly explained: "Please note that there are two methods to identify alternate settings: the number, which is the value in bAlternateSetting, and the index, which is the index in the descriptor array. There might be some wording in the USB spec that these two values must be the same, but in reality, [insert standard rant about firmware writers], bAlternateSetting must be treated as a random ID value." This patch changes the name to express the correct usage semantics. No functional change. Signed-off-by: Eldad Zack <eldad@fogrefinery.com> Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/usb')
-rw-r--r--sound/usb/card.h2
-rw-r--r--sound/usb/endpoint.c6
-rw-r--r--sound/usb/pcm.c6
3 files changed, 7 insertions, 7 deletions
diff --git a/sound/usb/card.h b/sound/usb/card.h
index ca98a9b915c9..9867ab866857 100644
--- a/sound/usb/card.h
+++ b/sound/usb/card.h
@@ -96,7 +96,7 @@ struct snd_usb_endpoint {
96 unsigned int syncinterval; /* P for adaptive mode, 0 otherwise */ 96 unsigned int syncinterval; /* P for adaptive mode, 0 otherwise */
97 unsigned char silence_value; 97 unsigned char silence_value;
98 unsigned int stride; 98 unsigned int stride;
99 int iface, alt_idx; 99 int iface, altsetting;
100 int skip_packets; /* quirks for devices to ignore the first n packets 100 int skip_packets; /* quirks for devices to ignore the first n packets
101 in a stream */ 101 in a stream */
102 102
diff --git a/sound/usb/endpoint.c b/sound/usb/endpoint.c
index 26856601422b..e25e32a8824d 100644
--- a/sound/usb/endpoint.c
+++ b/sound/usb/endpoint.c
@@ -426,9 +426,9 @@ struct snd_usb_endpoint *snd_usb_add_endpoint(struct snd_usb_audio *chip,
426 list_for_each_entry(ep, &chip->ep_list, list) { 426 list_for_each_entry(ep, &chip->ep_list, list) {
427 if (ep->ep_num == ep_num && 427 if (ep->ep_num == ep_num &&
428 ep->iface == alts->desc.bInterfaceNumber && 428 ep->iface == alts->desc.bInterfaceNumber &&
429 ep->alt_idx == alts->desc.bAlternateSetting) { 429 ep->altsetting == alts->desc.bAlternateSetting) {
430 snd_printdd(KERN_DEBUG "Re-using EP %x in iface %d,%d @%p\n", 430 snd_printdd(KERN_DEBUG "Re-using EP %x in iface %d,%d @%p\n",
431 ep_num, ep->iface, ep->alt_idx, ep); 431 ep_num, ep->iface, ep->altsetting, ep);
432 goto __exit_unlock; 432 goto __exit_unlock;
433 } 433 }
434 } 434 }
@@ -447,7 +447,7 @@ struct snd_usb_endpoint *snd_usb_add_endpoint(struct snd_usb_audio *chip,
447 ep->type = type; 447 ep->type = type;
448 ep->ep_num = ep_num; 448 ep->ep_num = ep_num;
449 ep->iface = alts->desc.bInterfaceNumber; 449 ep->iface = alts->desc.bInterfaceNumber;
450 ep->alt_idx = alts->desc.bAlternateSetting; 450 ep->altsetting = alts->desc.bAlternateSetting;
451 INIT_LIST_HEAD(&ep->ready_playback_urbs); 451 INIT_LIST_HEAD(&ep->ready_playback_urbs);
452 ep_num &= USB_ENDPOINT_NUMBER_MASK; 452 ep_num &= USB_ENDPOINT_NUMBER_MASK;
453 453
diff --git a/sound/usb/pcm.c b/sound/usb/pcm.c
index 0c1a95dd5538..ca3256d6fde3 100644
--- a/sound/usb/pcm.c
+++ b/sound/usb/pcm.c
@@ -241,17 +241,17 @@ static int start_endpoints(struct snd_usb_substream *subs, bool can_sleep)
241 struct snd_usb_endpoint *ep = subs->sync_endpoint; 241 struct snd_usb_endpoint *ep = subs->sync_endpoint;
242 242
243 if (subs->data_endpoint->iface != subs->sync_endpoint->iface || 243 if (subs->data_endpoint->iface != subs->sync_endpoint->iface ||
244 subs->data_endpoint->alt_idx != subs->sync_endpoint->alt_idx) { 244 subs->data_endpoint->altsetting != subs->sync_endpoint->altsetting) {
245 err = usb_set_interface(subs->dev, 245 err = usb_set_interface(subs->dev,
246 subs->sync_endpoint->iface, 246 subs->sync_endpoint->iface,
247 subs->sync_endpoint->alt_idx); 247 subs->sync_endpoint->altsetting);
248 if (err < 0) { 248 if (err < 0) {
249 clear_bit(SUBSTREAM_FLAG_SYNC_EP_STARTED, &subs->flags); 249 clear_bit(SUBSTREAM_FLAG_SYNC_EP_STARTED, &subs->flags);
250 snd_printk(KERN_ERR 250 snd_printk(KERN_ERR
251 "%d:%d:%d: cannot set interface (%d)\n", 251 "%d:%d:%d: cannot set interface (%d)\n",
252 subs->dev->devnum, 252 subs->dev->devnum,
253 subs->sync_endpoint->iface, 253 subs->sync_endpoint->iface,
254 subs->sync_endpoint->alt_idx, err); 254 subs->sync_endpoint->altsetting, err);
255 return -EIO; 255 return -EIO;
256 } 256 }
257 } 257 }