brcmfmac: fix send_pkts statistic counter in firmware-signalling
authorArend van Spriel <arend@broadcom.com>
Thu, 6 Jun 2013 11:18:02 +0000 (13:18 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 12 Jun 2013 19:02:21 +0000 (15:02 -0400)
The statistic counter send_pkts was wrongly counted conditionally.
Correcting the mistake.

Reviewed-by: Hante Meuleman <meuleman@broadcom.com>
Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/brcm80211/brcmfmac/fwsignal.c

index dc40eec8a5ccba3bac2d88cb2ac3072aee98ffd9..04aa4b286611f327b7a60a0f2350fab438c5d124 100644 (file)
@@ -1728,10 +1728,9 @@ static int brcmf_fws_commit_skb(struct brcmf_fws_info *fws, int fifo,
                entry->suppr_transit_count++;
        entry->seq[fifo]++;
        fws->stats.pkt2bus++;
-       if (brcmf_skbcb(skb)->if_flags & BRCMF_SKB_IF_FLAGS_CREDITCHECK_MASK) {
-               fws->stats.send_pkts[fifo]++;
+       fws->stats.send_pkts[fifo]++;
+       if (brcmf_skbcb(skb)->if_flags & BRCMF_SKB_IF_FLAGS_CREDITCHECK_MASK)
                fws->stats.fifo_credits_sent[fifo]++;
-       }
 
        return rc;