local variable "ret" is not used. hence it is safe to remove and just return 0. Signed-off-by: zhong jiang <zhongjiang@xxxxxxxxxx> --- drivers/net/wireless/intel/iwlegacy/4965-mac.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/net/wireless/intel/iwlegacy/4965-mac.c b/drivers/net/wireless/intel/iwlegacy/4965-mac.c index 51fdd7c..3664f56 100644 --- a/drivers/net/wireless/intel/iwlegacy/4965-mac.c +++ b/drivers/net/wireless/intel/iwlegacy/4965-mac.c @@ -3331,7 +3331,6 @@ struct il_mod_params il4965_mod_params = { struct ieee80211_key_conf *keyconf, u8 sta_id) { unsigned long flags; - int ret = 0; __le16 key_flags = 0; key_flags |= (STA_KEY_FLG_TKIP | STA_KEY_FLG_MAP_KEY_MSK); @@ -3368,7 +3367,7 @@ struct il_mod_params il4965_mod_params = { spin_unlock_irqrestore(&il->sta_lock, flags); - return ret; + return 0; } void -- 1.7.12.4