Commit 51cbbb0e authored by Martin Kaiser's avatar Martin Kaiser Committed by Greg Kroah-Hartman
Browse files

staging: r8188eu: remove unnecessary rtw_free_xmitframe call 



There's no need to call rtw_free_xmitframe before we dequeue the first
frame. pxmitframe is always NULL at this point, rtw_free_xmitframe will do
nothing in this case.

Signed-off-by: default avatarMartin Kaiser <martin@kaiser.cx>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com> # Edimax N150
Link: https://lore.kernel.org/r/20221230180646.91008-18-martin@kaiser.cx


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 26cf6c2c

drivers/staging/r8188eu/hal/rtl8188eu_xmit.c

+0 −3
Original line number Diff line number Diff line
@@ -398,9 +398,6 @@ bool rtl8188eu_xmitframe_complete(struct adapter *adapt, struct xmit_priv *pxmit 
	if (!pxmitbuf)

		return false;



	/* 3 1. pick up first frame */

	rtw_free_xmitframe(pxmitpriv, pxmitframe);



	pxmitframe = rtw_dequeue_xframe(pxmitpriv, pxmitpriv->hwxmits, pxmitpriv->hwxmit_entry);

	if (!pxmitframe) {

		/*  no more xmit frame, release xmit buffer */