Hi Michał,
Thanks for the review.
On Mon, Aug 18, 2025 at 11:50:16AM +0200, Michał Pecio wrote:
> > @@ -1351,8 +1369,18 @@ static u32 xhci_get_endpoint_max_burst(struct usb_device *udev,
> >
> > if (udev->speed == USB_SPEED_HIGH &&
> > (usb_endpoint_xfer_isoc(&ep->desc) ||
> > - usb_endpoint_xfer_int(&ep->desc)))
> > + usb_endpoint_xfer_int(&ep->desc))) {
> > + /*
> > + * eUSB2 double isoc bw endpoints max packet field service
> > + * opportunity bits 12:11 are not valid, so set the ctx burst to
> > + * max service opportunity "2" as these eps support transferring
> > + * over 3072 bytes per interval
> > + */
>
> I think a shorter comment would suffice: eUSB2 BWD uses fixed burst
> size and max packets bits 12:11 are invalid.
I'll use this:
+ * eUSB2 double isochronous BW ECN uses fixed burst size and max
+ * packets bits 12:11 are invalid.
--
Sakari Ailus