diff options
author | Emil Tantilov <emil.s.tantilov@intel.com> | 2010-07-27 02:37:21 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-07-27 02:37:21 -0400 |
commit | feb8f47809fcc60250d28a6ddabc0ddbe9360d7c (patch) | |
tree | 0d4d6b59328786dc057afec7d469fd2875adb6b7 /drivers/net/e1000/e1000_ethtool.c | |
parent | 66cc42a4bc23a5f621407d1c23b9fe29d41c92c6 (diff) |
e1000: use netif_<level> instead of netdev_<level>
This patch restores the ability to set msglvl through ethtool.
The issue was introduced by:
commit 675ad47375c76a7c3be4ace9554d92cd55518ced
Reported-by: Joe Perches <joe@perches.com>
Signed-off-by: Emil Tantilov <emil.s.tantilov@intel.com>
Tested-by: Jeff Pieper <jeffrey.e.pieper@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/e1000/e1000_ethtool.c')
-rw-r--r-- | drivers/net/e1000/e1000_ethtool.c | 27 |
1 files changed, 14 insertions, 13 deletions
diff --git a/drivers/net/e1000/e1000_ethtool.c b/drivers/net/e1000/e1000_ethtool.c index d5ff029aa7b2..f4d0922ec65b 100644 --- a/drivers/net/e1000/e1000_ethtool.c +++ b/drivers/net/e1000/e1000_ethtool.c | |||
@@ -346,7 +346,7 @@ static int e1000_set_tso(struct net_device *netdev, u32 data) | |||
346 | 346 | ||
347 | netdev->features &= ~NETIF_F_TSO6; | 347 | netdev->features &= ~NETIF_F_TSO6; |
348 | 348 | ||
349 | e_info("TSO is %s\n", data ? "Enabled" : "Disabled"); | 349 | e_info(probe, "TSO is %s\n", data ? "Enabled" : "Disabled"); |
350 | adapter->tso_force = true; | 350 | adapter->tso_force = true; |
351 | return 0; | 351 | return 0; |
352 | } | 352 | } |
@@ -714,9 +714,9 @@ static bool reg_pattern_test(struct e1000_adapter *adapter, u64 *data, int reg, | |||
714 | writel(write & test[i], address); | 714 | writel(write & test[i], address); |
715 | read = readl(address); | 715 | read = readl(address); |
716 | if (read != (write & test[i] & mask)) { | 716 | if (read != (write & test[i] & mask)) { |
717 | e_info("pattern test reg %04X failed: " | 717 | e_err(drv, "pattern test reg %04X failed: " |
718 | "got 0x%08X expected 0x%08X\n", | 718 | "got 0x%08X expected 0x%08X\n", |
719 | reg, read, (write & test[i] & mask)); | 719 | reg, read, (write & test[i] & mask)); |
720 | *data = reg; | 720 | *data = reg; |
721 | return true; | 721 | return true; |
722 | } | 722 | } |
@@ -734,7 +734,7 @@ static bool reg_set_and_check(struct e1000_adapter *adapter, u64 *data, int reg, | |||
734 | writel(write & mask, address); | 734 | writel(write & mask, address); |
735 | read = readl(address); | 735 | read = readl(address); |
736 | if ((read & mask) != (write & mask)) { | 736 | if ((read & mask) != (write & mask)) { |
737 | e_err("set/check reg %04X test failed: " | 737 | e_err(drv, "set/check reg %04X test failed: " |
738 | "got 0x%08X expected 0x%08X\n", | 738 | "got 0x%08X expected 0x%08X\n", |
739 | reg, (read & mask), (write & mask)); | 739 | reg, (read & mask), (write & mask)); |
740 | *data = reg; | 740 | *data = reg; |
@@ -779,7 +779,7 @@ static int e1000_reg_test(struct e1000_adapter *adapter, u64 *data) | |||
779 | ew32(STATUS, toggle); | 779 | ew32(STATUS, toggle); |
780 | after = er32(STATUS) & toggle; | 780 | after = er32(STATUS) & toggle; |
781 | if (value != after) { | 781 | if (value != after) { |
782 | e_err("failed STATUS register test got: " | 782 | e_err(drv, "failed STATUS register test got: " |
783 | "0x%08X expected: 0x%08X\n", after, value); | 783 | "0x%08X expected: 0x%08X\n", after, value); |
784 | *data = 1; | 784 | *data = 1; |
785 | return 1; | 785 | return 1; |
@@ -894,7 +894,8 @@ static int e1000_intr_test(struct e1000_adapter *adapter, u64 *data) | |||
894 | *data = 1; | 894 | *data = 1; |
895 | return -1; | 895 | return -1; |
896 | } | 896 | } |
897 | e_info("testing %s interrupt\n", (shared_int ? "shared" : "unshared")); | 897 | e_info(hw, "testing %s interrupt\n", (shared_int ? |
898 | "shared" : "unshared")); | ||
898 | 899 | ||
899 | /* Disable all the interrupts */ | 900 | /* Disable all the interrupts */ |
900 | ew32(IMC, 0xFFFFFFFF); | 901 | ew32(IMC, 0xFFFFFFFF); |
@@ -1561,7 +1562,7 @@ static void e1000_diag_test(struct net_device *netdev, | |||
1561 | u8 forced_speed_duplex = hw->forced_speed_duplex; | 1562 | u8 forced_speed_duplex = hw->forced_speed_duplex; |
1562 | u8 autoneg = hw->autoneg; | 1563 | u8 autoneg = hw->autoneg; |
1563 | 1564 | ||
1564 | e_info("offline testing starting\n"); | 1565 | e_info(hw, "offline testing starting\n"); |
1565 | 1566 | ||
1566 | /* Link test performed before hardware reset so autoneg doesn't | 1567 | /* Link test performed before hardware reset so autoneg doesn't |
1567 | * interfere with test result */ | 1568 | * interfere with test result */ |
@@ -1601,7 +1602,7 @@ static void e1000_diag_test(struct net_device *netdev, | |||
1601 | if (if_running) | 1602 | if (if_running) |
1602 | dev_open(netdev); | 1603 | dev_open(netdev); |
1603 | } else { | 1604 | } else { |
1604 | e_info("online testing starting\n"); | 1605 | e_info(hw, "online testing starting\n"); |
1605 | /* Online tests */ | 1606 | /* Online tests */ |
1606 | if (e1000_link_test(adapter, &data[4])) | 1607 | if (e1000_link_test(adapter, &data[4])) |
1607 | eth_test->flags |= ETH_TEST_FL_FAILED; | 1608 | eth_test->flags |= ETH_TEST_FL_FAILED; |
@@ -1694,8 +1695,8 @@ static void e1000_get_wol(struct net_device *netdev, | |||
1694 | wol->supported &= ~WAKE_UCAST; | 1695 | wol->supported &= ~WAKE_UCAST; |
1695 | 1696 | ||
1696 | if (adapter->wol & E1000_WUFC_EX) | 1697 | if (adapter->wol & E1000_WUFC_EX) |
1697 | e_err("Interface does not support " | 1698 | e_err(drv, "Interface does not support directed " |
1698 | "directed (unicast) frame wake-up packets\n"); | 1699 | "(unicast) frame wake-up packets\n"); |
1699 | break; | 1700 | break; |
1700 | default: | 1701 | default: |
1701 | break; | 1702 | break; |
@@ -1726,8 +1727,8 @@ static int e1000_set_wol(struct net_device *netdev, struct ethtool_wolinfo *wol) | |||
1726 | switch (hw->device_id) { | 1727 | switch (hw->device_id) { |
1727 | case E1000_DEV_ID_82546GB_QUAD_COPPER_KSP3: | 1728 | case E1000_DEV_ID_82546GB_QUAD_COPPER_KSP3: |
1728 | if (wol->wolopts & WAKE_UCAST) { | 1729 | if (wol->wolopts & WAKE_UCAST) { |
1729 | e_err("Interface does not support " | 1730 | e_err(drv, "Interface does not support directed " |
1730 | "directed (unicast) frame wake-up packets\n"); | 1731 | "(unicast) frame wake-up packets\n"); |
1731 | return -EOPNOTSUPP; | 1732 | return -EOPNOTSUPP; |
1732 | } | 1733 | } |
1733 | break; | 1734 | break; |