>-----Original Message----- >From: Arnd Bergmann [mailto:[email protected]] >Sent: Thursday, June 16, 2016 5:08 PM >To: Dept-GE Linux NIC Dev <[email protected]> >Cc: Arnd Bergmann <[email protected]>; David Miller ><[email protected]>; Rajesh Borundia <[email protected]>; >netdev <[email protected]>; linux-kernel <linux- >[email protected]> >Subject: [PATCH 1/5] net: qlcnic: don't set unused function argument > >We get a warning for qlcnic_83xx_get_mac_address when building with >"make W=1": > >drivers/net/ethernet/qlogic/qlcnic/qlcnic_83xx_hw.c: In function >'qlcnic_83xx_get_mac_address': >drivers/net/ethernet/qlogic/qlcnic/qlcnic_83xx_hw.c:2156:8: error: >parameter 'function' set but not used [-Werror=unused-but-set-parameter] > >Clearly this is harmless, but there is also no point for setting the variable, >so >we can simply remove the assignment. > >Signed-off-by: Arnd Bergmann <[email protected]> >--- > drivers/net/ethernet/qlogic/qlcnic/qlcnic_83xx_hw.c | 1 - > 1 file changed, 1 deletion(-) > >diff --git a/drivers/net/ethernet/qlogic/qlcnic/qlcnic_83xx_hw.c >b/drivers/net/ethernet/qlogic/qlcnic/qlcnic_83xx_hw.c >index f9640d5ce6ba..bdbcd2b088a0 100644 >--- a/drivers/net/ethernet/qlogic/qlcnic/qlcnic_83xx_hw.c >+++ b/drivers/net/ethernet/qlogic/qlcnic/qlcnic_83xx_hw.c >@@ -2159,7 +2159,6 @@ int qlcnic_83xx_get_mac_address(struct >qlcnic_adapter *adapter, u8 *mac, > struct qlcnic_cmd_args cmd; > u32 mac_low, mac_high; > >- function = 0; > err = qlcnic_alloc_mbx_args(&cmd, adapter, >QLCNIC_CMD_MAC_ADDRESS); > if (err) > return err; >-- >2.9.0
Acked-by: Rajesh Borundia <[email protected]>
