From: Takashi Iwai <tiwai@suse.de>
To: linux-sound@vger.kernel.org
Subject: [PATCH] ALSA: usb-audio: Simplify with usb_endpoint_max_periodic_payload()
Date: Tue, 2 Dec 2025 08:08:27 +0100 [thread overview]
Message-ID: <20251202070828.145656-1-tiwai@suse.de> (raw)
Recently we received a new helper function,
usb_endpoint_max_periodic_payload(), for calculating the max packet
size for periodic transfer.
Simplify the former open code with the new helper function.
Fixes: a748e1dbb2df ("ALSA: usb-audio: Fix max bytes-per-interval calculation")
Suggested-by: Michal Pecio <michal.pecio@gmail.com>
Link: https://lore.kernel.org/20251124210518.90054-1-dylan_robinson@motu.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
---
sound/usb/stream.c | 34 ++--------------------------------
1 file changed, 2 insertions(+), 32 deletions(-)
diff --git a/sound/usb/stream.c b/sound/usb/stream.c
index 074a61215de6..ec7d756d78d1 100644
--- a/sound/usb/stream.c
+++ b/sound/usb/stream.c
@@ -684,43 +684,13 @@ snd_usb_find_output_terminal_descriptor(struct usb_host_interface *ctrl_iface,
return NULL;
}
-static unsigned int
-snd_usb_max_bytes_per_interval(struct snd_usb_audio *chip,
- struct usb_host_interface *alts)
-{
- struct usb_host_endpoint *ep = &alts->endpoint[0];
- unsigned int max_bytes = usb_endpoint_maxp(&ep->desc);
-
- /* SuperSpeed isoc endpoints have up to 16 bursts of up to 3 packets each */
- if (snd_usb_get_speed(chip->dev) >= USB_SPEED_SUPER) {
- int burst = 1 + ep->ss_ep_comp.bMaxBurst;
- int mult = USB_SS_MULT(ep->ss_ep_comp.bmAttributes);
- max_bytes *= burst;
- max_bytes *= mult;
- }
-
- if (snd_usb_get_speed(chip->dev) == USB_SPEED_SUPER_PLUS &&
- USB_SS_SSP_ISOC_COMP(ep->ss_ep_comp.bmAttributes)) {
- max_bytes = le32_to_cpu(ep->ssp_isoc_ep_comp.dwBytesPerInterval);
- }
-
- /* High speed, 1-3 packets/uframe, max 6 for eUSB2 double bw */
- if (snd_usb_get_speed(chip->dev) == USB_SPEED_HIGH) {
- if (usb_endpoint_is_hs_isoc_double(chip->dev, ep))
- max_bytes = le32_to_cpu(ep->eusb2_isoc_ep_comp.dwBytesPerInterval);
- else
- max_bytes *= usb_endpoint_maxp_mult(&ep->desc);
- }
-
- return max_bytes;
-}
-
static struct audioformat *
audio_format_alloc_init(struct snd_usb_audio *chip,
struct usb_host_interface *alts,
int protocol, int iface_no, int altset_idx,
int altno, int num_channels, int clock)
{
+ struct usb_host_endpoint *ep = &alts->endpoint[0];
struct audioformat *fp;
fp = kzalloc(sizeof(*fp), GFP_KERNEL);
@@ -734,7 +704,7 @@ audio_format_alloc_init(struct snd_usb_audio *chip,
fp->ep_attr = get_endpoint(alts, 0)->bmAttributes;
fp->datainterval = snd_usb_parse_datainterval(chip, alts);
fp->protocol = protocol;
- fp->maxpacksize = snd_usb_max_bytes_per_interval(chip, alts);
+ fp->maxpacksize = usb_endpoint_max_periodic_payload(chip->dev, ep);
fp->channels = num_channels;
fp->clock = clock;
INIT_LIST_HEAD(&fp->list);
--
2.52.0
reply other threads:[~2025-12-02 7:08 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20251202070828.145656-1-tiwai@suse.de \
--to=tiwai@suse.de \
--cc=linux-sound@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox