e1000: mark unused parameters
authorIntel <intel.com>
Fri, 8 Nov 2013 02:00:00 +0000 (03:00 +0100)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Sun, 24 Nov 2013 00:31:33 +0000 (01:31 +0100)
Signed-off-by: Intel
lib/librte_pmd_e1000/e1000/e1000_defines.h
lib/librte_pmd_e1000/e1000/e1000_mac.c
lib/librte_pmd_e1000/e1000/e1000_mbx.c
lib/librte_pmd_e1000/e1000/e1000_nvm.c
lib/librte_pmd_e1000/e1000/e1000_osdep.h
lib/librte_pmd_e1000/e1000/e1000_phy.c
lib/librte_pmd_e1000/e1000/e1000_vf.c
lib/librte_pmd_e1000/em_rxtx.c

index c4f5aae..8f67050 100644 (file)
@@ -1887,4 +1887,7 @@ POSSIBILITY OF SUCH DAMAGE.
 /* Lan ID bit field offset in status register */
 #define E1000_STATUS_LAN_ID_OFFSET     2
 #define E1000_VFTA_ENTRIES             128
+#ifndef E1000_UNUSEDARG
+#define E1000_UNUSEDARG
+#endif /* E1000_UNUSEDARG */
 #endif /* _E1000_DEFINES_H_ */
index ac149ff..b779e31 100644 (file)
@@ -87,9 +87,10 @@ void e1000_init_mac_ops_generic(struct e1000_hw *hw)
  *  e1000_null_ops_generic - No-op function, returns 0
  *  @hw: pointer to the HW structure
  **/
-s32 e1000_null_ops_generic(struct e1000_hw *hw)
+s32 e1000_null_ops_generic(struct e1000_hw E1000_UNUSEDARG *hw)
 {
        DEBUGFUNC("e1000_null_ops_generic");
+       UNREFERENCED_1PARAMETER(hw);
        return E1000_SUCCESS;
 }
 
@@ -97,9 +98,10 @@ s32 e1000_null_ops_generic(struct e1000_hw *hw)
  *  e1000_null_mac_generic - No-op function, return void
  *  @hw: pointer to the HW structure
  **/
-void e1000_null_mac_generic(struct e1000_hw *hw)
+void e1000_null_mac_generic(struct e1000_hw E1000_UNUSEDARG *hw)
 {
        DEBUGFUNC("e1000_null_mac_generic");
+       UNREFERENCED_1PARAMETER(hw);
        return;
 }
 
@@ -107,9 +109,11 @@ void e1000_null_mac_generic(struct e1000_hw *hw)
  *  e1000_null_link_info - No-op function, return 0
  *  @hw: pointer to the HW structure
  **/
-s32 e1000_null_link_info(struct e1000_hw *hw, u16 *s, u16 *d)
+s32 e1000_null_link_info(struct e1000_hw E1000_UNUSEDARG *hw,
+                        u16 E1000_UNUSEDARG *s, u16 E1000_UNUSEDARG *d)
 {
        DEBUGFUNC("e1000_null_link_info");
+       UNREFERENCED_3PARAMETER(hw, s, d);
        return E1000_SUCCESS;
 }
 
@@ -117,9 +121,10 @@ s32 e1000_null_link_info(struct e1000_hw *hw, u16 *s, u16 *d)
  *  e1000_null_mng_mode - No-op function, return false
  *  @hw: pointer to the HW structure
  **/
-bool e1000_null_mng_mode(struct e1000_hw *hw)
+bool e1000_null_mng_mode(struct e1000_hw E1000_UNUSEDARG *hw)
 {
        DEBUGFUNC("e1000_null_mng_mode");
+       UNREFERENCED_1PARAMETER(hw);
        return false;
 }
 
@@ -127,9 +132,11 @@ bool e1000_null_mng_mode(struct e1000_hw *hw)
  *  e1000_null_update_mc - No-op function, return void
  *  @hw: pointer to the HW structure
  **/
-void e1000_null_update_mc(struct e1000_hw *hw, u8 *h, u32 a)
+void e1000_null_update_mc(struct e1000_hw E1000_UNUSEDARG *hw,
+                         u8 E1000_UNUSEDARG *h, u32 E1000_UNUSEDARG a)
 {
        DEBUGFUNC("e1000_null_update_mc");
+       UNREFERENCED_3PARAMETER(hw, h, a);
        return;
 }
 
@@ -137,9 +144,11 @@ void e1000_null_update_mc(struct e1000_hw *hw, u8 *h, u32 a)
  *  e1000_null_write_vfta - No-op function, return void
  *  @hw: pointer to the HW structure
  **/
-void e1000_null_write_vfta(struct e1000_hw *hw, u32 a, u32 b)
+void e1000_null_write_vfta(struct e1000_hw E1000_UNUSEDARG *hw,
+                          u32 E1000_UNUSEDARG a, u32 E1000_UNUSEDARG b)
 {
        DEBUGFUNC("e1000_null_write_vfta");
+       UNREFERENCED_3PARAMETER(hw, a, b);
        return;
 }
 
@@ -147,9 +156,11 @@ void e1000_null_write_vfta(struct e1000_hw *hw, u32 a, u32 b)
  *  e1000_null_rar_set - No-op function, return void
  *  @hw: pointer to the HW structure
  **/
-void e1000_null_rar_set(struct e1000_hw *hw, u8 *h, u32 a)
+void e1000_null_rar_set(struct e1000_hw E1000_UNUSEDARG *hw,
+                       u8 E1000_UNUSEDARG *h, u32 E1000_UNUSEDARG a)
 {
        DEBUGFUNC("e1000_null_rar_set");
+       UNREFERENCED_3PARAMETER(hw, h, a);
        return;
 }
 
@@ -1668,10 +1679,11 @@ s32 e1000_get_speed_and_duplex_copper_generic(struct e1000_hw *hw, u16 *speed,
  *  Sets the speed and duplex to gigabit full duplex (the only possible option)
  *  for fiber/serdes links.
  **/
-s32 e1000_get_speed_and_duplex_fiber_serdes_generic(struct e1000_hw *hw,
+s32 e1000_get_speed_and_duplex_fiber_serdes_generic(struct e1000_hw E1000_UNUSEDARG *hw,
                                                    u16 *speed, u16 *duplex)
 {
        DEBUGFUNC("e1000_get_speed_and_duplex_fiber_serdes_generic");
+       UNREFERENCED_1PARAMETER(hw);
 
        *speed = SPEED_1000;
        *duplex = FULL_DUPLEX;
@@ -2168,9 +2180,10 @@ STATIC s32 e1000_validate_mdi_setting_generic(struct e1000_hw *hw)
  *  Validate the MDI/MDIx setting, allowing for auto-crossover during forced
  *  operation.
  **/
-s32 e1000_validate_mdi_setting_crossover_generic(struct e1000_hw *hw)
+s32 e1000_validate_mdi_setting_crossover_generic(struct e1000_hw E1000_UNUSEDARG *hw)
 {
        DEBUGFUNC("e1000_validate_mdi_setting_crossover_generic");
+       UNREFERENCED_1PARAMETER(hw);
 
        return E1000_SUCCESS;
 }
index cc95fb6..5ada971 100644 (file)
@@ -37,9 +37,11 @@ POSSIBILITY OF SUCH DAMAGE.
  *  e1000_null_mbx_check_for_flag - No-op function, return 0
  *  @hw: pointer to the HW structure
  **/
-STATIC s32 e1000_null_mbx_check_for_flag(struct e1000_hw *hw, u16 mbx_id)
+STATIC s32 e1000_null_mbx_check_for_flag(struct e1000_hw E1000_UNUSEDARG *hw,
+                                        u16 E1000_UNUSEDARG mbx_id)
 {
        DEBUGFUNC("e1000_null_mbx_check_flag");
+       UNREFERENCED_2PARAMETER(hw, mbx_id);
 
        return E1000_SUCCESS;
 }
@@ -48,10 +50,13 @@ STATIC s32 e1000_null_mbx_check_for_flag(struct e1000_hw *hw, u16 mbx_id)
  *  e1000_null_mbx_transact - No-op function, return 0
  *  @hw: pointer to the HW structure
  **/
-STATIC s32 e1000_null_mbx_transact(struct e1000_hw *hw, u32 *msg, u16 size,
-                                  u16 mbx_id)
+STATIC s32 e1000_null_mbx_transact(struct e1000_hw E1000_UNUSEDARG *hw,
+                                  u32 E1000_UNUSEDARG *msg,
+                                  u16 E1000_UNUSEDARG size,
+                                  u16 E1000_UNUSEDARG mbx_id)
 {
        DEBUGFUNC("e1000_null_mbx_rw_msg");
+       UNREFERENCED_4PARAMETER(hw, msg, size, mbx_id);
 
        return E1000_SUCCESS;
 }
@@ -353,10 +358,12 @@ STATIC s32 e1000_check_for_bit_vf(struct e1000_hw *hw, u32 mask)
  *
  *  returns SUCCESS if the PF has set the Status bit or else ERR_MBX
  **/
-STATIC s32 e1000_check_for_msg_vf(struct e1000_hw *hw, u16 mbx_id)
+STATIC s32 e1000_check_for_msg_vf(struct e1000_hw *hw,
+                                 u16 E1000_UNUSEDARG mbx_id)
 {
        s32 ret_val = -E1000_ERR_MBX;
 
+       UNREFERENCED_1PARAMETER(mbx_id);
        DEBUGFUNC("e1000_check_for_msg_vf");
 
        if (!e1000_check_for_bit_vf(hw, E1000_V2PMAILBOX_PFSTS)) {
@@ -374,10 +381,12 @@ STATIC s32 e1000_check_for_msg_vf(struct e1000_hw *hw, u16 mbx_id)
  *
  *  returns SUCCESS if the PF has set the ACK bit or else ERR_MBX
  **/
-STATIC s32 e1000_check_for_ack_vf(struct e1000_hw *hw, u16 mbx_id)
+STATIC s32 e1000_check_for_ack_vf(struct e1000_hw *hw,
+                                 u16 E1000_UNUSEDARG mbx_id)
 {
        s32 ret_val = -E1000_ERR_MBX;
 
+       UNREFERENCED_1PARAMETER(mbx_id);
        DEBUGFUNC("e1000_check_for_ack_vf");
 
        if (!e1000_check_for_bit_vf(hw, E1000_V2PMAILBOX_PFACK)) {
@@ -395,10 +404,12 @@ STATIC s32 e1000_check_for_ack_vf(struct e1000_hw *hw, u16 mbx_id)
  *
  *  returns true if the PF has set the reset done bit or else false
  **/
-STATIC s32 e1000_check_for_rst_vf(struct e1000_hw *hw, u16 mbx_id)
+STATIC s32 e1000_check_for_rst_vf(struct e1000_hw *hw,
+                                 u16 E1000_UNUSEDARG mbx_id)
 {
        s32 ret_val = -E1000_ERR_MBX;
 
+       UNREFERENCED_1PARAMETER(mbx_id);
        DEBUGFUNC("e1000_check_for_rst_vf");
 
        if (!e1000_check_for_bit_vf(hw, (E1000_V2PMAILBOX_RSTD |
@@ -442,11 +453,12 @@ STATIC s32 e1000_obtain_mbx_lock_vf(struct e1000_hw *hw)
  *  returns SUCCESS if it successfully copied message into the buffer
  **/
 STATIC s32 e1000_write_mbx_vf(struct e1000_hw *hw, u32 *msg, u16 size,
-                             u16 mbx_id)
+                             u16 E1000_UNUSEDARG mbx_id)
 {
        s32 ret_val;
        u16 i;
 
+       UNREFERENCED_1PARAMETER(mbx_id);
 
        DEBUGFUNC("e1000_write_mbx_vf");
 
@@ -483,12 +495,13 @@ out_no_write:
  *  returns SUCCESS if it successfuly read message from buffer
  **/
 STATIC s32 e1000_read_mbx_vf(struct e1000_hw *hw, u32 *msg, u16 size,
-                            u16 mbx_id)
+                            u16 E1000_UNUSEDARG mbx_id)
 {
        s32 ret_val = E1000_SUCCESS;
        u16 i;
 
        DEBUGFUNC("e1000_read_mbx_vf");
+       UNREFERENCED_1PARAMETER(mbx_id);
 
        /* lock the mailbox to prevent pf/vf race condition */
        ret_val = e1000_obtain_mbx_lock_vf(hw);
index 52ba68f..79146c4 100644 (file)
@@ -62,9 +62,12 @@ void e1000_init_nvm_ops_generic(struct e1000_hw *hw)
  *  e1000_null_nvm_read - No-op function, return 0
  *  @hw: pointer to the HW structure
  **/
-s32 e1000_null_read_nvm(struct e1000_hw *hw, u16 a, u16 b, u16 *c)
+s32 e1000_null_read_nvm(struct e1000_hw E1000_UNUSEDARG *hw,
+                       u16 E1000_UNUSEDARG a, u16 E1000_UNUSEDARG b,
+                       u16 E1000_UNUSEDARG *c)
 {
        DEBUGFUNC("e1000_null_read_nvm");
+       UNREFERENCED_4PARAMETER(hw, a, b, c);
        return E1000_SUCCESS;
 }
 
@@ -72,9 +75,10 @@ s32 e1000_null_read_nvm(struct e1000_hw *hw, u16 a, u16 b, u16 *c)
  *  e1000_null_nvm_generic - No-op function, return void
  *  @hw: pointer to the HW structure
  **/
-void e1000_null_nvm_generic(struct e1000_hw *hw)
+void e1000_null_nvm_generic(struct e1000_hw E1000_UNUSEDARG *hw)
 {
        DEBUGFUNC("e1000_null_nvm_generic");
+       UNREFERENCED_1PARAMETER(hw);
        return;
 }
 
@@ -82,9 +86,11 @@ void e1000_null_nvm_generic(struct e1000_hw *hw)
  *  e1000_null_led_default - No-op function, return 0
  *  @hw: pointer to the HW structure
  **/
-s32 e1000_null_led_default(struct e1000_hw *hw, u16 *data)
+s32 e1000_null_led_default(struct e1000_hw E1000_UNUSEDARG *hw,
+                          u16 E1000_UNUSEDARG *data)
 {
        DEBUGFUNC("e1000_null_led_default");
+       UNREFERENCED_2PARAMETER(hw, data);
        return E1000_SUCCESS;
 }
 
@@ -92,9 +98,12 @@ s32 e1000_null_led_default(struct e1000_hw *hw, u16 *data)
  *  e1000_null_write_nvm - No-op function, return 0
  *  @hw: pointer to the HW structure
  **/
-s32 e1000_null_write_nvm(struct e1000_hw *hw, u16 a, u16 b, u16 *c)
+s32 e1000_null_write_nvm(struct e1000_hw E1000_UNUSEDARG *hw,
+                        u16 E1000_UNUSEDARG a, u16 E1000_UNUSEDARG b,
+                        u16 E1000_UNUSEDARG *c)
 {
        DEBUGFUNC("e1000_null_write_nvm");
+       UNREFERENCED_4PARAMETER(hw, a, b, c);
        return E1000_SUCCESS;
 }
 
index a5b50aa..cfea424 100644 (file)
 #define DEBUGOUT6(S, args...)   DEBUGOUT(S, ##args)
 #define DEBUGOUT7(S, args...)   DEBUGOUT(S, ##args)
 
+#define UNREFERENCED_PARAMETER(_p)
+#define UNREFERENCED_1PARAMETER(_p)
+#define UNREFERENCED_2PARAMETER(_p, _q)
+#define UNREFERENCED_3PARAMETER(_p, _q, _r)
+#define UNREFERENCED_4PARAMETER(_p, _q, _r, _s)
+
 #define FALSE                  0
 #define TRUE                   1
 
index dad0202..0b77809 100644 (file)
@@ -102,9 +102,11 @@ void e1000_init_phy_ops_generic(struct e1000_hw *hw)
  *  e1000_null_set_page - No-op function, return 0
  *  @hw: pointer to the HW structure
  **/
-s32 e1000_null_set_page(struct e1000_hw *hw, u16 data)
+s32 e1000_null_set_page(struct e1000_hw E1000_UNUSEDARG *hw,
+                       u16 E1000_UNUSEDARG data)
 {
        DEBUGFUNC("e1000_null_set_page");
+       UNREFERENCED_2PARAMETER(hw, data);
        return E1000_SUCCESS;
 }
 
@@ -112,9 +114,11 @@ s32 e1000_null_set_page(struct e1000_hw *hw, u16 data)
  *  e1000_null_read_reg - No-op function, return 0
  *  @hw: pointer to the HW structure
  **/
-s32 e1000_null_read_reg(struct e1000_hw *hw, u32 offset, u16 *data)
+s32 e1000_null_read_reg(struct e1000_hw E1000_UNUSEDARG *hw,
+                       u32 E1000_UNUSEDARG offset, u16 E1000_UNUSEDARG *data)
 {
        DEBUGFUNC("e1000_null_read_reg");
+       UNREFERENCED_3PARAMETER(hw, offset, data);
        return E1000_SUCCESS;
 }
 
@@ -122,9 +126,10 @@ s32 e1000_null_read_reg(struct e1000_hw *hw, u32 offset, u16 *data)
  *  e1000_null_phy_generic - No-op function, return void
  *  @hw: pointer to the HW structure
  **/
-void e1000_null_phy_generic(struct e1000_hw *hw)
+void e1000_null_phy_generic(struct e1000_hw E1000_UNUSEDARG *hw)
 {
        DEBUGFUNC("e1000_null_phy_generic");
+       UNREFERENCED_1PARAMETER(hw);
        return;
 }
 
@@ -132,9 +137,11 @@ void e1000_null_phy_generic(struct e1000_hw *hw)
  *  e1000_null_lplu_state - No-op function, return 0
  *  @hw: pointer to the HW structure
  **/
-s32 e1000_null_lplu_state(struct e1000_hw *hw, bool active)
+s32 e1000_null_lplu_state(struct e1000_hw E1000_UNUSEDARG *hw,
+                         bool E1000_UNUSEDARG active)
 {
        DEBUGFUNC("e1000_null_lplu_state");
+       UNREFERENCED_2PARAMETER(hw, active);
        return E1000_SUCCESS;
 }
 
@@ -142,9 +149,11 @@ s32 e1000_null_lplu_state(struct e1000_hw *hw, bool active)
  *  e1000_null_write_reg - No-op function, return 0
  *  @hw: pointer to the HW structure
  **/
-s32 e1000_null_write_reg(struct e1000_hw *hw, u32 offset, u16 data)
+s32 e1000_null_write_reg(struct e1000_hw E1000_UNUSEDARG *hw,
+                        u32 E1000_UNUSEDARG offset, u16 E1000_UNUSEDARG data)
 {
        DEBUGFUNC("e1000_null_write_reg");
+       UNREFERENCED_3PARAMETER(hw, offset, data);
        return E1000_SUCCESS;
 }
 
@@ -156,10 +165,13 @@ s32 e1000_null_write_reg(struct e1000_hw *hw, u32 offset, u16 data)
  *  @data: data value read
  *
  **/
-s32 e1000_read_i2c_byte_null(struct e1000_hw *hw, u8 byte_offset,
-                            u8 dev_addr, u8 *data)
+s32 e1000_read_i2c_byte_null(struct e1000_hw E1000_UNUSEDARG *hw,
+                            u8 E1000_UNUSEDARG byte_offset,
+                            u8 E1000_UNUSEDARG dev_addr,
+                            u8 E1000_UNUSEDARG *data)
 {
        DEBUGFUNC("e1000_read_i2c_byte_null");
+       UNREFERENCED_4PARAMETER(hw, byte_offset, dev_addr, data);
        return E1000_SUCCESS;
 }
 
@@ -171,10 +183,13 @@ s32 e1000_read_i2c_byte_null(struct e1000_hw *hw, u8 byte_offset,
  *  @data: data value to write
  *
  **/
-s32 e1000_write_i2c_byte_null(struct e1000_hw *hw, u8 byte_offset,
-                             u8 dev_addr, u8 data)
+s32 e1000_write_i2c_byte_null(struct e1000_hw E1000_UNUSEDARG *hw,
+                             u8 E1000_UNUSEDARG byte_offset,
+                             u8 E1000_UNUSEDARG dev_addr,
+                             u8 E1000_UNUSEDARG data)
 {
        DEBUGFUNC("e1000_write_i2c_byte_null");
+       UNREFERENCED_4PARAMETER(hw, byte_offset, dev_addr, data);
        return E1000_SUCCESS;
 }
 
@@ -2843,9 +2858,10 @@ s32 e1000_phy_hw_reset_generic(struct e1000_hw *hw)
  *  Generic function to wait 10 milli-seconds for configuration to complete
  *  and return success.
  **/
-s32 e1000_get_cfg_done_generic(struct e1000_hw *hw)
+s32 e1000_get_cfg_done_generic(struct e1000_hw E1000_UNUSEDARG *hw)
 {
        DEBUGFUNC("e1000_get_cfg_done_generic");
+       UNREFERENCED_1PARAMETER(hw);
 
        msec_delay_irq(10);
 
index 7f1cd81..6f19f19 100644 (file)
@@ -158,8 +158,9 @@ void e1000_init_function_pointers_vf(struct e1000_hw *hw)
  *  In addition, the MAC registers to access PHY/NVM don't exist so we don't
  *  even want any SW to attempt to use them.
  **/
-STATIC s32 e1000_acquire_vf(struct e1000_hw *hw)
+STATIC s32 e1000_acquire_vf(struct e1000_hw E1000_UNUSEDARG *hw)
 {
+       UNREFERENCED_1PARAMETER(hw);
        return -E1000_ERR_PHY;
 }
 
@@ -171,8 +172,9 @@ STATIC s32 e1000_acquire_vf(struct e1000_hw *hw)
  *  In addition, the MAC registers to access PHY/NVM don't exist so we don't
  *  even want any SW to attempt to use them.
  **/
-STATIC void e1000_release_vf(struct e1000_hw *hw)
+STATIC void e1000_release_vf(struct e1000_hw E1000_UNUSEDARG *hw)
 {
+       UNREFERENCED_1PARAMETER(hw);
        return;
 }
 
@@ -182,9 +184,10 @@ STATIC void e1000_release_vf(struct e1000_hw *hw)
  *
  *  Virtual functions cannot change link.
  **/
-STATIC s32 e1000_setup_link_vf(struct e1000_hw *hw)
+STATIC s32 e1000_setup_link_vf(struct e1000_hw E1000_UNUSEDARG *hw)
 {
        DEBUGFUNC("e1000_setup_link_vf");
+       UNREFERENCED_1PARAMETER(hw);
 
        return E1000_SUCCESS;
 }
@@ -319,13 +322,15 @@ STATIC s32 e1000_init_hw_vf(struct e1000_hw *hw)
  *  @addr: pointer to the receive address
  *  @index receive address array register
  **/
-STATIC void e1000_rar_set_vf(struct e1000_hw *hw, u8 * addr, u32 index)
+STATIC void e1000_rar_set_vf(struct e1000_hw *hw, u8 *addr,
+                            u32 E1000_UNUSEDARG index)
 {
        struct e1000_mbx_info *mbx = &hw->mbx;
        u32 msgbuf[3];
        u8 *msg_addr = (u8 *)(&msgbuf[1]);
        s32 ret_val;
 
+       UNREFERENCED_1PARAMETER(index);
        memset(msgbuf, 0, 12);
        msgbuf[0] = E1000_VF_SET_MAC_ADDR;
        memcpy(msg_addr, addr, 6);
index c049542..0fade1b 100644 (file)
@@ -74,6 +74,7 @@
 #include "e1000_logs.h"
 #include "e1000/e1000_api.h"
 #include "e1000_ethdev.h"
+#include "e1000/e1000_osdep.h"
 
 #define        E1000_TXD_VLAN_SHIFT    16