Commit 31fa86cb authored Oct 29, 2015 by Hante Meuleman Committed by Kalle Valo Nov 26, 2015

brcmfmac: Remove unnecessary check from start_xmit.



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>

parent c9c00438

drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c

+0 −8

Original line number	Diff line number	Diff line
		@@ -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;

Admin message