Patchwork net: iwlegacy: remove unused variable

login
register
mail settings
Submitter Guenter Roeck
Date Jan. 27, 2013, 6:55 p.m.
Message ID <1359312941-30661-1-git-send-email-linux@roeck-us.net>
Download mbox | patch
Permalink /patch/216029/
State Not Applicable
Delegated to: David Miller
Headers show

Comments

Guenter Roeck - Jan. 27, 2013, 6:55 p.m.
Fix:

drivers/net/wireless/iwlegacy/4965.c: In function ‘il4965_post_associate’:
drivers/net/wireless/iwlegacy/4965.c:1751:25: warning: variable ‘conf’ set but
not used [-Wunused-but-set-variable]

seen when building allmodconfig on x86_64 with W=1 by removing the unused
variable.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>
---
 drivers/net/wireless/iwlegacy/4965.c |    3 ---
 1 file changed, 3 deletions(-)
Stanislaw Gruszka - Jan. 28, 2013, 9:56 a.m.
On Sun, Jan 27, 2013 at 10:55:41AM -0800, Guenter Roeck wrote:
> drivers/net/wireless/iwlegacy/4965.c: In function ‘il4965_post_associate’:
> drivers/net/wireless/iwlegacy/4965.c:1751:25: warning: variable ‘conf’ set but
> not used [-Wunused-but-set-variable]
> 
> seen when building allmodconfig on x86_64 with W=1 by removing the unused
> variable.
> 
> Signed-off-by: Guenter Roeck <linux@roeck-us.net>

ACK
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/drivers/net/wireless/iwlegacy/4965.c b/drivers/net/wireless/iwlegacy/4965.c
index 5db1171..91eb2d0 100644
--- a/drivers/net/wireless/iwlegacy/4965.c
+++ b/drivers/net/wireless/iwlegacy/4965.c
@@ -1748,7 +1748,6 @@  static void
 il4965_post_associate(struct il_priv *il)
 {
 	struct ieee80211_vif *vif = il->vif;
-	struct ieee80211_conf *conf = NULL;
 	int ret = 0;
 
 	if (!vif || !il->is_open)
@@ -1759,8 +1758,6 @@  il4965_post_associate(struct il_priv *il)
 
 	il_scan_cancel_timeout(il, 200);
 
-	conf = &il->hw->conf;
-
 	il->staging.filter_flags &= ~RXON_FILTER_ASSOC_MSK;
 	il_commit_rxon(il);