brcm80211: fmac: use counters in brcmf_bus structure
authorArend van Spriel <arend@broadcom.com>
Fri, 2 Mar 2012 21:55:47 +0000 (22:55 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 5 Mar 2012 20:53:55 +0000 (15:53 -0500)
The usb code defines a structure for counting statistics. However,
it should use the statistics entry provided in brcmf_bus as that is
exposed to the net_device. The usb private statistics counter only
remains with counters for control packets between driver and usb
device.

Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Reviewed-by: Franky (Zhenhui) Lin <frankyl@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/usb.c
drivers/net/wireless/brcm80211/brcmfmac/usb.h

index 0418a40fce8d848a9915276294af501dd68c73f3..a2a0c45201acd49e7cc9fd727c561392f4151647 100644 (file)
@@ -514,9 +514,9 @@ static void brcmf_usb_tx_complete(struct urb *urb)
 
        brcmf_usb_del_fromq(devinfo, req);
        if (urb->status == 0)
-               devinfo->bus_pub.stats.tx_packets++;
+               devinfo->bus_pub.bus->dstats.tx_packets++;
        else
-               devinfo->bus_pub.stats.tx_errors++;
+               devinfo->bus_pub.bus->dstats.tx_errors++;
 
        dev_kfree_skb(req->skb);
        req->skb = NULL;
@@ -536,9 +536,9 @@ static void brcmf_usb_rx_complete(struct urb *urb)
        req->skb = NULL;
 
        if (urb->status == 0) {
-               devinfo->bus_pub.stats.rx_packets++;
+               devinfo->bus_pub.bus->dstats.rx_packets++;
        } else {
-               devinfo->bus_pub.stats.rx_errors++;
+               devinfo->bus_pub.bus->dstats.rx_errors++;
                dev_kfree_skb(skb);
                brcmf_usb_enq(devinfo, &devinfo->rx_freeq, req);
                return;
index 3377d63e6c8b6f222c5f7eeb5156aa0e7658546b..acfa5e89872fb8c8103ebb626beef5c0ba0064a2 100644 (file)
@@ -33,21 +33,10 @@ enum brcmf_usb_pnp_state {
 };
 
 struct brcmf_stats {
-       u32 tx_errors;
-       u32 tx_packets;
-       u32 tx_multicast;
        u32 tx_ctlpkts;
        u32 tx_ctlerrs;
-       u32 tx_dropped;
-       u32 tx_flushed;
-       u32 rx_errors;
-       u32 rx_packets;
-       u32 rx_multicast;
        u32 rx_ctlpkts;
        u32 rx_ctlerrs;
-       u32 rx_dropped;
-       u32 rx_flushed;
-
 };
 
 struct brcmf_usbdev {