Commit 5bd9e94a authored by Dan Carpenter's avatar Dan Carpenter Committed by Greg Kroah-Hartman
Browse files

staging: rtl8188eu: delete some dead code 



Calling rtw_free_xmitframe() with a NULL "pxmitframe" parameter is a
no-op.  It appears that originally this code was part of a loop but it
was already dead code by the time that the driver was merged into the
kernel.

Fixes: 7bc88639 ("staging: r8188eu: Add files for new driver - part 17")
Reviewed-by: default avatarGuenter Roeck <linux@roeck-us.net>
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Link: https://lore.kernel.org/r/YL5jFF9ldj0W9P1f@mwanda


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

drivers/staging/rtl8188eu/hal/rtl8188eu_xmit.c

+1 −4
Original line number Diff line number Diff line
@@ -413,8 +413,7 @@ static u32 xmitframe_need_length(struct xmit_frame *pxmitframe) 
bool rtl8188eu_xmitframe_complete(struct adapter *adapt,

				  struct xmit_priv *pxmitpriv)

{

	struct xmit_frame *pxmitframe = NULL;

	struct xmit_frame *n;

	struct xmit_frame *pxmitframe, *n;

	struct xmit_frame *pfirstframe = NULL;

	struct xmit_buf *pxmitbuf;
@@ -443,8 +442,6 @@ bool rtl8188eu_xmitframe_complete(struct adapter *adapt, 
		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 */