staging: r8188eu: bkq_cnt is write-only
authorMartin Kaiser <martin@kaiser.cx>
Mon, 9 Jan 2023 21:28:42 +0000 (22:28 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 17 Jan 2023 18:35:24 +0000 (19:35 +0100)
bkq_cnt in struct xmit_priv is initialised, incremented and decremented
but never read. Remove bkq_cnt and resulting dead code.

Signed-off-by: Martin Kaiser <martin@kaiser.cx>
Acked-by: Pavel Skripkin <paskripkin@gmail.com>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com> # Edimax N150
Link: https://lore.kernel.org/r/20230109212852.75612-3-martin@kaiser.cx
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/r8188eu/core/rtw_xmit.c
drivers/staging/r8188eu/include/rtw_xmit.h
drivers/staging/r8188eu/os_dep/usb_ops_linux.c

index 47b339b..c06ab9b 100644 (file)
@@ -195,7 +195,6 @@ int _rtw_init_xmit_priv(struct xmit_priv *pxmitpriv, struct adapter *padapter)
        pxmitpriv->txirp_cnt = 1;
 
        /* per AC pending irp */
-       pxmitpriv->bkq_cnt = 0;
        pxmitpriv->viq_cnt = 0;
        pxmitpriv->voq_cnt = 0;
 
index a4d645a..8defce2 100644 (file)
@@ -283,7 +283,6 @@ struct      xmit_priv {
        u8              txirp_cnt;/*  */
        struct tasklet_struct xmit_tasklet;
        /* per AC pending irp */
-       int bkq_cnt;
        int viq_cnt;
        int voq_cnt;
        struct __queue free_xmitbuf_queue;
index 067b7de..5ae09ed 100644 (file)
@@ -48,9 +48,6 @@ static void usb_write_port_complete(struct urb *purb, struct pt_regs *regs)
        case VI_QUEUE_INX:
                pxmitpriv->viq_cnt--;
                break;
-       case BK_QUEUE_INX:
-               pxmitpriv->bkq_cnt--;
-               break;
        case HIGH_QUEUE_INX:
                rtw_chk_hi_queue_cmd(padapter);
                break;
@@ -123,7 +120,6 @@ u32 rtw_write_port(struct adapter *padapter, u32 addr, u32 cnt, u8 *wmem)
                pxmitbuf->flags = BE_QUEUE_INX;
                break;
        case BK_QUEUE_INX:
-               pxmitpriv->bkq_cnt++;
                pxmitbuf->flags = BK_QUEUE_INX;
                break;
        case HIGH_QUEUE_INX: