staging: r8188eu: remove function rtw_hal_free_xmit_priv
authorPhillip Potter <phil@philpotter.co.uk>
Wed, 18 Aug 2021 23:42:52 +0000 (00:42 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 19 Aug 2021 05:42:07 +0000 (07:42 +0200)
Remove function rtw_hal_free_xmit_priv in hal/hal_intf.c and its
declaration in include/hal_intf.h, as well as the single call line in
core/rtw_xmit.c. This function now essentially checks a function
pointer which will always be NULL (due to previous patches) and
therefore will never be called.

Acked-by: Fabio M. De Francesco <fmdefrancesco@gmail.com>
Signed-off-by: Phillip Potter <phil@philpotter.co.uk>
Link: https://lore.kernel.org/r/20210818234253.208271-6-phil@philpotter.co.uk
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/r8188eu/core/rtw_xmit.c
drivers/staging/r8188eu/hal/hal_intf.c
drivers/staging/r8188eu/include/hal_intf.h

index cdfcf94..69f997f 100644 (file)
@@ -220,8 +220,6 @@ void _rtw_free_xmit_priv(struct xmit_priv *pxmitpriv)
        u32 max_xmit_extbuf_size = MAX_XMIT_EXTBUF_SZ;
        u32 num_xmit_extbuf = NR_XMIT_EXTBUFF;
 
-       rtw_hal_free_xmit_priv(padapter);
-
        rtw_mfree_xmit_priv_lock(pxmitpriv);
 
        if (!pxmitpriv->pxmit_frame_buf)
index 47af081..a6d589e 100644 (file)
@@ -226,12 +226,6 @@ s32 rtw_hal_init_xmit_priv(struct adapter *adapt)
        return _FAIL;
 }
 
-void rtw_hal_free_xmit_priv(struct adapter *adapt)
-{
-       if (adapt->HalFunc.free_xmit_priv)
-               adapt->HalFunc.free_xmit_priv(adapt);
-}
-
 s32 rtw_hal_init_recv_priv(struct adapter *adapt)
 {
        if (adapt->HalFunc.init_recv_priv)
index 37ddb97..8a24200 100644 (file)
@@ -356,7 +356,6 @@ s32 rtw_hal_mgnt_xmit(struct adapter *padapter,
                          struct xmit_frame *pmgntframe);
 
 s32    rtw_hal_init_xmit_priv(struct adapter *padapter);
-void   rtw_hal_free_xmit_priv(struct adapter *padapter);
 
 s32    rtw_hal_init_recv_priv(struct adapter *padapter);
 void   rtw_hal_free_recv_priv(struct adapter *padapter);