staging: et131x: Remove some unecessary packet accounting code

The ce_stats ipackets and opackets members are only used to update
the net_device_stats->[r,t]x_packets counters, so get rid of them.

Signed-off-by: Mark Einon <mark.einon@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

diff --git a/drivers/staging/et131x/et1310_rx.c b/drivers/staging/et131x/et1310_rx.c
index 6c0a64bdd593c0547cbcfd44f24ecb14d65847c0..fc90096f19c601e62bc644ee5ae0b9b579ecaa68 100644 (file)
--- a/drivers/staging/et131x/et1310_rx.c
+++ b/drivers/staging/et131x/et1310_rx.c
@@ -1131,7 +1131,7 @@ void et131x_handle_recv_interrupt(struct et131x_adapter *etdev)
                        continue;
 
                /* Increment the number of packets we received */
-               etdev->stats.ipackets++;
+               etdev->net_stats.rx_packets++;
 
                /* Set the status on the packet, either resources or success */
                if (etdev->rx_ring.nReadyRecv < RFD_LOW_WATER_MARK) {

diff --git a/drivers/staging/et131x/et1310_tx.c b/drivers/staging/et131x/et1310_tx.c
index b8f8dae57d6c7c578b5b938940de2e7c9363bb3c..bb67192270f1b71729bbe46e62871466291f1311 100644 (file)
--- a/drivers/staging/et131x/et1310_tx.c
+++ b/drivers/staging/et131x/et1310_tx.c
@@ -673,7 +673,7 @@ inline void et131x_free_send_packet(struct et131x_adapter *etdev,
        /* Add the TCB to the Ready Q */
        spin_lock_irqsave(&etdev->TCBReadyQLock, flags);
 
-       etdev->stats.opackets++;
+       etdev->net_stats.tx_packets++;
 
        if (etdev->tx_ring.tcb_qtail)
                etdev->tx_ring.tcb_qtail->next = tcb;

diff --git a/drivers/staging/et131x/et131x_adapter.h b/drivers/staging/et131x/et131x_adapter.h
index efa90a0eadc450bc3ab2cdd54031879e78599e8e..77164a69977750f8987522d9936df1ac5f87be73 100644 (file)
--- a/drivers/staging/et131x/et131x_adapter.h
+++ b/drivers/staging/et131x/et131x_adapter.h
@@ -99,10 +99,6 @@ struct rfd {
 
 /* Struct to define some device statistics */
 struct ce_stats {
-       /* Link Input/Output stats */
-       uint64_t ipackets;      /* # of in packets */
-       uint64_t opackets;      /* # of out packets */
-
        /* MIB II variables
         *
         * NOTE: atomic_t types are only guaranteed to store 24-bits; if we
@@ -242,7 +238,6 @@ struct et131x_adapter {
        struct ce_stats stats;
 
        struct net_device_stats net_stats;
-       struct net_device_stats net_stats_prev;
 };
 
 #endif /* __ET131X_ADAPTER_H__ */

diff --git a/drivers/staging/et131x/et131x_netdev.c b/drivers/staging/et131x/et131x_netdev.c
index c890d095730afa19a5b7a78307ac0cf372688027..6bcddf1fe587b266c2736623de02e4c1141d865d 100644 (file)
--- a/drivers/staging/et131x/et131x_netdev.c
+++ b/drivers/staging/et131x/et131x_netdev.c
@@ -88,22 +88,18 @@
 #include "et131x_adapter.h"
 #include "et131x.h"
 
-struct net_device_stats *et131x_stats(struct net_device *netdev);
-
 /**
  * et131x_stats - Return the current device statistics.
  * @netdev: device whose stats are being queried
  *
  * Returns 0 on success, errno on failure (as defined in errno.h)
  */
-struct net_device_stats *et131x_stats(struct net_device *netdev)
+static struct net_device_stats *et131x_stats(struct net_device *netdev)
 {
        struct et131x_adapter *adapter = netdev_priv(netdev);
        struct net_device_stats *stats = &adapter->net_stats;
        struct ce_stats *devstat = &adapter->stats;
 
-       stats->rx_packets = devstat->ipackets;
-       stats->tx_packets = devstat->opackets;
        stats->rx_errors = devstat->length_err + devstat->alignment_err +
            devstat->crc_err + devstat->code_violations + devstat->other_errors;
        stats->tx_errors = devstat->max_pkt_error;