RE: [RFC: 2.6 patch] drivers/net/igb/: #if 0 unused functions

From: Duyck, Alexander H
Date: Wed Jul 23 2008 - 13:27:51 EST


With the exception of the igb_mta_set function these functions can be
removed. I will try to work with Jeff Kirsher to have a pair of patches
submitted to clean up the igb_mta_set function for correct use and
remove the extra functions.

Thanks,

Alex

-----Original Message-----
From: Adrian Bunk [mailto:bunk@xxxxxxxxxx]
Sent: Tuesday, July 22, 2008 10:20 AM
To: Duyck, Alexander H; Kok, Auke-jan H; Kirsher, Jeffrey T
Cc: e1000-devel@xxxxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
Subject: [RFC: 2.6 patch] drivers/net/igb/: #if 0 unused functions

In commit 2d064c06fecadadcb81a452acd373af00dfb1fec
(igb: add 82576 MAC support) the following functions
became added unused resp. lost all of their users:
- e1000_82575.c: igb_translate_register_82576()
- e1000_mac.c: igb_init_rx_addrs()
- e1000_mac.c: igb_mta_set()
- e1000_mac.c: igb_update_mc_addr_list()

Signed-off-by: Adrian Bunk <bunk@xxxxxxxxxx>

---

drivers/net/igb/e1000_82575.c | 2 ++
drivers/net/igb/e1000_82575.h | 1 -
drivers/net/igb/e1000_mac.c | 6 ++++++
drivers/net/igb/e1000_mac.h | 4 ----
4 files changed, 8 insertions(+), 5 deletions(-)

00659c33ed602bd9efd271b74c691dc88260e6e9 Revert "Revert """
diff --git a/drivers/net/igb/e1000_82575.c
b/drivers/net/igb/e1000_82575.c
index e098f23..a4028a9 100644
--- a/drivers/net/igb/e1000_82575.c
+++ b/drivers/net/igb/e1000_82575.c
@@ -1231,6 +1231,7 @@ out:
return ret_val;
}

+#if 0
/**
* igb_translate_register_82576 - Translate the proper register offset
* @reg: e1000 register to be read
@@ -1294,6 +1295,7 @@ u32 igb_translate_register_82576(u32 reg)

return reg;
}
+#endif /* 0 */

/**
* igb_reset_init_script_82575 - Inits HW defaults after reset
diff --git a/drivers/net/igb/e1000_82575.h
b/drivers/net/igb/e1000_82575.h
index 2f848e5..c1928b5 100644
--- a/drivers/net/igb/e1000_82575.h
+++ b/drivers/net/igb/e1000_82575.h
@@ -28,7 +28,6 @@
#ifndef _E1000_82575_H_
#define _E1000_82575_H_

-u32 igb_translate_register_82576(u32 reg);
void igb_update_mc_addr_list_82575(struct e1000_hw*, u8*, u32, u32,
u32);
extern void igb_shutdown_fiber_serdes_link_82575(struct e1000_hw *hw);
extern void igb_rx_fifo_flush_82575(struct e1000_hw *hw);
diff --git a/drivers/net/igb/e1000_mac.c b/drivers/net/igb/e1000_mac.c
index 20408aa..a35c0d2 100644
--- a/drivers/net/igb/e1000_mac.c
+++ b/drivers/net/igb/e1000_mac.c
@@ -143,6 +143,7 @@ void igb_write_vfta(struct e1000_hw *hw, u32 offset,
u32 value)
wrfl();
}

+#if 0
/**
* igb_init_rx_addrs - Initialize receive address's
* @hw: pointer to the HW structure
@@ -170,6 +171,7 @@ void igb_init_rx_addrs(struct e1000_hw *hw, u16
rar_count)
wrfl();
}
}
+#endif /* 0 */

/**
* igb_check_alt_mac_addr - Check for alternate MAC addr
@@ -261,6 +263,8 @@ void igb_rar_set(struct e1000_hw *hw, u8 *addr, u32
index)
array_wr32(E1000_RA, ((index << 1) + 1), rar_high);
}

+#if 0
+
/**
* igb_mta_set - Set multicast filter table address
* @hw: pointer to the HW structure
@@ -350,6 +354,8 @@ void igb_update_mc_addr_list(struct e1000_hw *hw,
}
}

+#endif /* 0 */
+
/**
* igb_hash_mc_addr - Generate a multicast hash value
* @hw: pointer to the HW structure
diff --git a/drivers/net/igb/e1000_mac.h b/drivers/net/igb/e1000_mac.h
index dc2f8cc..5143fed 100644
--- a/drivers/net/igb/e1000_mac.h
+++ b/drivers/net/igb/e1000_mac.h
@@ -51,9 +51,6 @@ s32 igb_get_speed_and_duplex_copper(struct e1000_hw
*hw, u16 *speed,
u16 *duplex);
s32 igb_id_led_init(struct e1000_hw *hw);
s32 igb_led_off(struct e1000_hw *hw);
-void igb_update_mc_addr_list(struct e1000_hw *hw,
- u8 *mc_addr_list, u32 mc_addr_count,
- u32 rar_used_count, u32 rar_count);
s32 igb_setup_link(struct e1000_hw *hw);
s32 igb_validate_mdi_setting(struct e1000_hw *hw);
s32 igb_write_8bit_ctrl_reg(struct e1000_hw *hw, u32 reg,
@@ -62,7 +59,6 @@ s32 igb_write_8bit_ctrl_reg(struct e1000_hw *hw, u32
reg,
void igb_clear_hw_cntrs_base(struct e1000_hw *hw);
void igb_clear_vfta(struct e1000_hw *hw);
void igb_config_collision_dist(struct e1000_hw *hw);
-void igb_init_rx_addrs(struct e1000_hw *hw, u16 rar_count);
void igb_put_hw_semaphore(struct e1000_hw *hw);
void igb_rar_set(struct e1000_hw *hw, u8 *addr, u32 index);
s32 igb_check_alt_mac_addr(struct e1000_hw *hw);

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/