wifi: libertas_tf: refactor endpoint lookup

Use the common USB helpers for looking up bulk and interrupt endpoints
(and determining max packet size) instead of open coding.

Note that the driver has an implicit max packet size check which is
kept.

Signed-off-by: Johan Hovold <johan@kernel.org>
Link: https://patch.msgid.link/20260407151111.3187826-4-johan@kernel.org
Signed-off-by: Johannes Berg <johannes.berg@intel.com>

diff --git a/drivers/net/wireless/marvell/libertas_tf/if_usb.c b/drivers/net/wireless/marvell/libertas_tf/if_usb.c
index 07b38f2b8f58270163d864b80d2cb530f9ad0b7e..b85c6d783bf7b282ca27f8ff5061dfa6f437dd70 100644 (file)
--- a/drivers/net/wireless/marvell/libertas_tf/if_usb.c
+++ b/drivers/net/wireless/marvell/libertas_tf/if_usb.c
@@ -144,12 +144,12 @@ static const struct lbtf_ops if_usb_ops = {
 static int if_usb_probe(struct usb_interface *intf,
                        const struct usb_device_id *id)
 {
+       struct usb_endpoint_descriptor *ep_in, *ep_out;
        struct usb_device *udev;
        struct usb_host_interface *iface_desc;
-       struct usb_endpoint_descriptor *endpoint;
        struct lbtf_private *priv;
        struct if_usb_card *cardp;
-       int i;
+       int ret;
 
        lbtf_deb_enter(LBTF_DEB_USB);
        udev = interface_to_usbdev(intf);
@@ -171,31 +171,27 @@ static int if_usb_probe(struct usb_interface *intf,
                     udev->descriptor.bDeviceSubClass,
                     udev->descriptor.bDeviceProtocol);
 
-       for (i = 0; i < iface_desc->desc.bNumEndpoints; ++i) {
-               endpoint = &iface_desc->endpoint[i].desc;
-               if (usb_endpoint_is_bulk_in(endpoint)) {
-                       cardp->ep_in_size =
-                               le16_to_cpu(endpoint->wMaxPacketSize);
-                       cardp->ep_in = usb_endpoint_num(endpoint);
-
-                       lbtf_deb_usbd(&udev->dev, "in_endpoint = %d\n",
-                               cardp->ep_in);
-                       lbtf_deb_usbd(&udev->dev, "Bulk in size is %d\n",
-                               cardp->ep_in_size);
-               } else if (usb_endpoint_is_bulk_out(endpoint)) {
-                       cardp->ep_out_size =
-                               le16_to_cpu(endpoint->wMaxPacketSize);
-                       cardp->ep_out = usb_endpoint_num(endpoint);
-
-                       lbtf_deb_usbd(&udev->dev, "out_endpoint = %d\n",
-                               cardp->ep_out);
-                       lbtf_deb_usbd(&udev->dev, "Bulk out size is %d\n",
-                               cardp->ep_out_size);
-               }
+       ret = usb_find_common_endpoints_reverse(iface_desc, &ep_in, &ep_out,
+                                               NULL, NULL);
+       if (ret) {
+               lbtf_deb_usbd(&udev->dev, "Endpoints not found\n");
+               goto dealloc;
        }
+
+       cardp->ep_in_size = usb_endpoint_maxp(ep_in);
+       cardp->ep_in = usb_endpoint_num(ep_in);
+
+       lbtf_deb_usbd(&udev->dev, "in_endpoint = %d\n", cardp->ep_in);
+       lbtf_deb_usbd(&udev->dev, "Bulk in size is %d\n", cardp->ep_in_size);
+
+       cardp->ep_out_size = usb_endpoint_maxp(ep_out);
+       cardp->ep_out = usb_endpoint_num(ep_out);
+
+       lbtf_deb_usbd(&udev->dev, "out_endpoint = %d\n", cardp->ep_out);
+       lbtf_deb_usbd(&udev->dev, "Bulk out size is %d\n", cardp->ep_out_size);
+
        if (!cardp->ep_out_size || !cardp->ep_in_size) {
-               lbtf_deb_usbd(&udev->dev, "Endpoints not found\n");
-               /* Endpoints not found */
+               lbtf_deb_usbd(&udev->dev, "Endpoints not valid\n");
                goto dealloc;
        }