brcmfmac: Remove unnecessary check from start_xmit.
authorHante Meuleman <meuleman@broadcom.com>
Thu, 29 Oct 2015 19:33:13 +0000 (20:33 +0100)
committerKalle Valo <kvalo@codeaurora.org>
Thu, 26 Nov 2015 11:53:15 +0000 (13:53 +0200)
The brcmf_netdev_start_xmit checks if the ndev is still valid by
checking if it still exists in database. This check is not needed
and therefor removed.

Reviewed-by: Arend Van Spriel <arend@broadcom.com>
Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Signed-off-by: Hante Meuleman <meuleman@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c

index 76b66bc15b598c11d58aa1c3bcba17bf84916b4b..b743a83fd2349b4c74a42e2e1e06279e7af929aa 100644 (file)
@@ -217,14 +217,6 @@ static netdev_tx_t brcmf_netdev_start_xmit(struct sk_buff *skb,
                goto done;
        }
 
-       if (!drvr->iflist[ifp->bssidx]) {
-               brcmf_err("bad ifidx %d\n", ifp->bssidx);
-               netif_stop_queue(ndev);
-               dev_kfree_skb(skb);
-               ret = -ENODEV;
-               goto done;
-       }
-
        /* Make sure there's enough room for any header */
        if (skb_headroom(skb) < drvr->hdrlen) {
                struct sk_buff *skb2;