RE: [PATCH 1/5] net: qlcnic: don't set unused function argument
From: Rajesh Borundia
Date: Thu Jun 16 2016 - 09:59:51 EST
>-----Original Message-----
>From: Arnd Bergmann [mailto:arnd@xxxxxxxx]
>Sent: Thursday, June 16, 2016 5:08 PM
>To: Dept-GE Linux NIC Dev <Dept-GELinuxNICDev@xxxxxxxxxx>
>Cc: Arnd Bergmann <arnd@xxxxxxxx>; David Miller
><davem@xxxxxxxxxxxxx>; Rajesh Borundia <rajesh.borundia@xxxxxxxxxx>;
>netdev <netdev@xxxxxxxxxxxxxxx>; linux-kernel <linux-
>kernel@xxxxxxxxxxxxxxx>
>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 <arnd@xxxxxxxx>
>---
> 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 <rajesh.borundia@xxxxxxxxxx>