staging: et131x: Remove ununsed statistics
authorMark Einon <mark.einon@gmail.com>
Thu, 11 Sep 2014 21:59:42 +0000 (22:59 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 19 Sep 2014 23:02:12 +0000 (16:02 -0700)
From struct ce_stats; unicast_pkts_rcvd, unicast_pkts_xmtd,
multicast_pkts_xmtd, broadcast_pkts_rcvd and broadcast_pkts_xmtd
are not returned or used for anything meaningful - remove the code
that collects them, and the struct members too.

Signed-off-by: Mark Einon <mark.einon@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/et131x/et131x.c

index 565d552..d37708c 100644 (file)
@@ -433,12 +433,7 @@ struct ce_stats {
         * MUST have 32, then we'll need another way to perform atomic
         * operations
         */
-       u32             unicast_pkts_rcvd;
-       atomic_t        unicast_pkts_xmtd;
        u32             multicast_pkts_rcvd;
-       atomic_t        multicast_pkts_xmtd;
-       u32             broadcast_pkts_rcvd;
-       atomic_t        broadcast_pkts_xmtd;
        u32             rcvd_pkts_dropped;
 
        /* Tx Statistics. */
@@ -2507,14 +2502,6 @@ static struct rfd *nic_rx_pkts(struct et131x_adapter *adapter)
 
                if (len > 0)
                        adapter->stats.multicast_pkts_rcvd++;
-       } else if (word0 & ALCATEL_BROADCAST_PKT) {
-               adapter->stats.broadcast_pkts_rcvd++;
-       } else {
-               /* Not sure what this counter measures in promiscuous mode.
-                * Perhaps we should check the MAC address to see if it is
-                * directed to us in promiscuous mode.
-                */
-               adapter->stats.unicast_pkts_rcvd++;
        }
 
        if (!len) {
@@ -2942,13 +2929,6 @@ static inline void free_send_packet(struct et131x_adapter *adapter,
        struct tx_ring *tx_ring = &adapter->tx_ring;
        u64  dma_addr;
 
-       if (tcb->flags & FMP_DEST_BROAD)
-               atomic_inc(&adapter->stats.broadcast_pkts_xmtd);
-       else if (tcb->flags & FMP_DEST_MULTI)
-               atomic_inc(&adapter->stats.multicast_pkts_xmtd);
-       else
-               atomic_inc(&adapter->stats.unicast_pkts_xmtd);
-
        if (tcb->skb) {
                stats->tx_bytes += tcb->skb->len;