summaryrefslogtreecommitdiff
path: root/drivers/net/wireless/ath/ath6kl
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2015-09-07 16:49:05 +0300
committerKalle Valo <kvalo@qca.qualcomm.com>2015-10-09 11:44:47 +0300
commit101d1f7fbf7dbb881f808c0fdc23124051ac5da6 (patch)
tree4e409242b6856c7c776a8c5ee7c6df287186b948 /drivers/net/wireless/ath/ath6kl
parentf13e0630826f961732f3192282a6c249bb6208c1 (diff)
downloadlinux-101d1f7fbf7dbb881f808c0fdc23124051ac5da6.tar.xz
ath6kl: remove redundant null pointer check on send_pkt
The check for send_pkt being NULL is redundant before the call to htc_reclaim_txctrl_buf, therefore it should be removed. This was detected by static analysis by cppcheck. Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
Diffstat (limited to 'drivers/net/wireless/ath/ath6kl')
-rw-r--r--drivers/net/wireless/ath/ath6kl/htc_mbox.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/net/wireless/ath/ath6kl/htc_mbox.c b/drivers/net/wireless/ath/ath6kl/htc_mbox.c
index e481f14b9878..fffb65b3e652 100644
--- a/drivers/net/wireless/ath/ath6kl/htc_mbox.c
+++ b/drivers/net/wireless/ath/ath6kl/htc_mbox.c
@@ -1085,9 +1085,7 @@ static int htc_setup_tx_complete(struct htc_target *target)
send_pkt->completion = NULL;
ath6kl_htc_tx_prep_pkt(send_pkt, 0, 0, 0);
status = ath6kl_htc_tx_issue(target, send_pkt);
-
- if (send_pkt != NULL)
- htc_reclaim_txctrl_buf(target, send_pkt);
+ htc_reclaim_txctrl_buf(target, send_pkt);
return status;
}