staging: r8188eu: don't store nr_endpoint in a global struct
authorMartin Kaiser <martin@kaiser.cx>
Fri, 26 Nov 2021 17:32:04 +0000 (18:32 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 3 Dec 2021 14:11:33 +0000 (15:11 +0100)
There's no need to store the number of endpoints in a global struct.
We only need it at startup when we iterate over the endpoints.

Signed-off-by: Martin Kaiser <martin@kaiser.cx>
Link: https://lore.kernel.org/r/20211126173205.21352-8-martin@kaiser.cx
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/r8188eu/include/drv_types.h
drivers/staging/r8188eu/os_dep/usb_intf.c

index a3fb509..1eb6b5f 100644 (file)
@@ -145,7 +145,6 @@ struct dvobj_priv {
 
 /*-------- below is for USB INTERFACE --------*/
 
-       u8      nr_endpoint;
        u8      ishighspeed;
        u8      RtNumInPipes;
        u8      RtNumOutPipes;
index e0a35a6..a00ae7a 100644 (file)
@@ -97,9 +97,8 @@ static struct dvobj_priv *usb_dvobj_init(struct usb_interface *usb_intf)
 
        pdvobjpriv->NumInterfaces = pconf_desc->bNumInterfaces;
        pdvobjpriv->InterfaceNumber = piface_desc->bInterfaceNumber;
-       pdvobjpriv->nr_endpoint = piface_desc->bNumEndpoints;
 
-       for (i = 0; i < pdvobjpriv->nr_endpoint; i++) {
+       for (i = 0; i < piface_desc->bNumEndpoints; i++) {
                int ep_num;
                pendp_desc = &phost_iface->endpoint[i].desc;