diff mbox

[v2] staging: rtl8723au: fix sparse warning

Message ID 20141224055521.GA16313@epbyminw1710.minsk.epam.com (mailing list archive)
State Not Applicable
Headers show

Commit Message

Alexey Tulia Dec. 24, 2014, 5:55 a.m. UTC
drivers/staging/rtl8723au/core/rtw_xmit.c:2375 warning: symbol
'rtw_ack_tx_done23a' was not declared. Should it be static?

Function 'rtw_ack_tx_done23a' seems to be unused in current staging
code.

Signed-off-by: Aliaskei Tulia <Aliaksei_Tulia@epam.com>
---
 drivers/staging/rtl8723au/core/rtw_xmit.c | 9 ---------
 1 file changed, 9 deletions(-)

--
2.1.0

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Greg KH Dec. 24, 2014, 6:08 a.m. UTC | #1
On Wed, Dec 24, 2014 at 08:55:21AM +0300, Aliaskei Tulia wrote:
> drivers/staging/rtl8723au/core/rtw_xmit.c:2375 warning: symbol
> 'rtw_ack_tx_done23a' was not declared. Should it be static?
> 
> Function 'rtw_ack_tx_done23a' seems to be unused in current staging
> code.
> 
> Signed-off-by: Aliaskei Tulia <Aliaksei_Tulia@epam.com>

This email address doesn't match your From: line :(
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/staging/rtl8723au/core/rtw_xmit.c b/drivers/staging/rtl8723au/core/rtw_xmit.c
index 7a5e6bf..1c82dff 100644
--- a/drivers/staging/rtl8723au/core/rtw_xmit.c
+++ b/drivers/staging/rtl8723au/core/rtw_xmit.c
@@ -2372,12 +2372,3 @@  int rtw_ack_tx_wait23a(struct xmit_priv *pxmitpriv, u32 timeout_ms)
 	return rtw_sctx_wait23a(pack_tx_ops);
 }

-void rtw_ack_tx_done23a(struct xmit_priv *pxmitpriv, int status)
-{
-	struct submit_ctx *pack_tx_ops = &pxmitpriv->ack_tx_ops;
-
-	if (pxmitpriv->ack_tx)
-		rtw23a_sctx_done_err(&pack_tx_ops, status);
-	else
-		DBG_8723A("%s ack_tx not set\n", __func__);
-}