diff options
author | Bruce Allan <bruce.w.allan@intel.com> | 2009-11-20 18:27:21 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-11-21 14:34:21 -0500 |
commit | 99673d9b5d48c81f2e9fe094c0d9e42815c60b3f (patch) | |
tree | e21e8f5149bb8a08497d97c3f0691dc81dbf2ac3 /drivers/net/e1000e/82571.c | |
parent | 3d5e33c9783d3e911e9aef0339663e887044f0df (diff) |
e1000e: cleanup functions that clear hardware statistics
The e1000_clear_hw_cntrs_*() functions read the registers to clear them.
There is no reason to save the register contents so the temp variable can
be removed.
Signed-off-by: Bruce Allan <bruce.w.allan@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/e1000e/82571.c')
-rw-r--r-- | drivers/net/e1000e/82571.c | 70 |
1 files changed, 34 insertions, 36 deletions
diff --git a/drivers/net/e1000e/82571.c b/drivers/net/e1000e/82571.c index 468dd7d4fcad..c5dc1b3be230 100644 --- a/drivers/net/e1000e/82571.c +++ b/drivers/net/e1000e/82571.c | |||
@@ -1613,44 +1613,42 @@ static s32 e1000_fix_nvm_checksum_82571(struct e1000_hw *hw) | |||
1613 | **/ | 1613 | **/ |
1614 | static void e1000_clear_hw_cntrs_82571(struct e1000_hw *hw) | 1614 | static void e1000_clear_hw_cntrs_82571(struct e1000_hw *hw) |
1615 | { | 1615 | { |
1616 | u32 temp; | ||
1617 | |||
1618 | e1000e_clear_hw_cntrs_base(hw); | 1616 | e1000e_clear_hw_cntrs_base(hw); |
1619 | 1617 | ||
1620 | temp = er32(PRC64); | 1618 | er32(PRC64); |
1621 | temp = er32(PRC127); | 1619 | er32(PRC127); |
1622 | temp = er32(PRC255); | 1620 | er32(PRC255); |
1623 | temp = er32(PRC511); | 1621 | er32(PRC511); |
1624 | temp = er32(PRC1023); | 1622 | er32(PRC1023); |
1625 | temp = er32(PRC1522); | 1623 | er32(PRC1522); |
1626 | temp = er32(PTC64); | 1624 | er32(PTC64); |
1627 | temp = er32(PTC127); | 1625 | er32(PTC127); |
1628 | temp = er32(PTC255); | 1626 | er32(PTC255); |
1629 | temp = er32(PTC511); | 1627 | er32(PTC511); |
1630 | temp = er32(PTC1023); | 1628 | er32(PTC1023); |
1631 | temp = er32(PTC1522); | 1629 | er32(PTC1522); |
1632 | 1630 | ||
1633 | temp = er32(ALGNERRC); | 1631 | er32(ALGNERRC); |
1634 | temp = er32(RXERRC); | 1632 | er32(RXERRC); |
1635 | temp = er32(TNCRS); | 1633 | er32(TNCRS); |
1636 | temp = er32(CEXTERR); | 1634 | er32(CEXTERR); |
1637 | temp = er32(TSCTC); | 1635 | er32(TSCTC); |
1638 | temp = er32(TSCTFC); | 1636 | er32(TSCTFC); |
1639 | 1637 | ||
1640 | temp = er32(MGTPRC); | 1638 | er32(MGTPRC); |
1641 | temp = er32(MGTPDC); | 1639 | er32(MGTPDC); |
1642 | temp = er32(MGTPTC); | 1640 | er32(MGTPTC); |
1643 | 1641 | ||
1644 | temp = er32(IAC); | 1642 | er32(IAC); |
1645 | temp = er32(ICRXOC); | 1643 | er32(ICRXOC); |
1646 | 1644 | ||
1647 | temp = er32(ICRXPTC); | 1645 | er32(ICRXPTC); |
1648 | temp = er32(ICRXATC); | 1646 | er32(ICRXATC); |
1649 | temp = er32(ICTXPTC); | 1647 | er32(ICTXPTC); |
1650 | temp = er32(ICTXATC); | 1648 | er32(ICTXATC); |
1651 | temp = er32(ICTXQEC); | 1649 | er32(ICTXQEC); |
1652 | temp = er32(ICTXQMTC); | 1650 | er32(ICTXQMTC); |
1653 | temp = er32(ICRXDMTC); | 1651 | er32(ICRXDMTC); |
1654 | } | 1652 | } |
1655 | 1653 | ||
1656 | static struct e1000_mac_operations e82571_mac_ops = { | 1654 | static struct e1000_mac_operations e82571_mac_ops = { |