media: uvcvideo: Simplify uvc_endpoint_max_bpi()
authorRicardo Ribalda <ribalda@chromium.org>
Fri, 1 Apr 2022 17:24:36 +0000 (19:24 +0200)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Sun, 8 May 2022 05:10:06 +0000 (07:10 +0200)
The case USB_SPEED_WIRELESS and the default one were doing the same.

Also, make always use of usb_endpoint_maxp_mult, as it should have a
sane value, even for LOW speed and WIRELESS.

Signed-off-by: Ricardo Ribalda <ribalda@chromium.org>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/media/usb/uvc/uvc_video.c

index e016f88..6712982 100644 (file)
@@ -1758,21 +1758,14 @@ static unsigned int uvc_endpoint_max_bpi(struct usb_device *dev,
                                         struct usb_host_endpoint *ep)
 {
        u16 psize;
-       u16 mult;
 
        switch (dev->speed) {
        case USB_SPEED_SUPER:
        case USB_SPEED_SUPER_PLUS:
                return le16_to_cpu(ep->ss_ep_comp.wBytesPerInterval);
-       case USB_SPEED_HIGH:
-               psize = usb_endpoint_maxp(&ep->desc);
-               mult = usb_endpoint_maxp_mult(&ep->desc);
-               return psize * mult;
-       case USB_SPEED_WIRELESS:
-               psize = usb_endpoint_maxp(&ep->desc);
-               return psize;
        default:
                psize = usb_endpoint_maxp(&ep->desc);
+               psize *= usb_endpoint_maxp_mult(&ep->desc);
                return psize;
        }
 }