staging: r8188eu: simplify the code to initialise inx
authorMartin Kaiser <martin@kaiser.cx>
Mon, 23 Jan 2023 20:53:25 +0000 (21:53 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 27 Jan 2023 09:10:35 +0000 (10:10 +0100)
Simplify the code to initialise the inx array in function
rtw_dequeue_xframe and make the code a tiny bit smaller.

Signed-off-by: Martin Kaiser <martin@kaiser.cx>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com> # Edimax N150
Link: https://lore.kernel.org/r/20230123205342.229589-7-martin@kaiser.cx
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/r8188eu/core/rtw_xmit.c

index b07a1ce21a3866210c5f8f25efd98952e161e28b..30bc9405356d2d32f87d4d45915a22c31e2c8dfc 100644 (file)
@@ -1384,15 +1384,11 @@ struct xmit_frame *rtw_dequeue_xframe(struct xmit_priv *pxmitpriv, struct hw_xmi
        struct xmit_frame *pxmitframe = NULL;
        struct adapter *padapter = pxmitpriv->adapter;
        struct registry_priv    *pregpriv = &padapter->registrypriv;
-       int i, inx[4];
-
-       inx[0] = 0; inx[1] = 1; inx[2] = 2; inx[3] = 3;
+       int i, inx[] = { 0, 1, 2, 3 };
 
        if (pregpriv->wifi_spec == 1) {
-               int j;
-
-               for (j = 0; j < 4; j++)
-                       inx[j] = pxmitpriv->wmm_para_seq[j];
+               for (i = 0; i < ARRAY_SIZE(inx); i++)
+                       inx[i] = pxmitpriv->wmm_para_seq[i];
        }
 
        spin_lock_bh(&pxmitpriv->lock);