From: Colin Ian King <colin.king@canonical.com>
Date: Wed, 27 Mar 2019 18:35:45 +0000 (+0000)
Subject: cfg80211: remove redundant zero check on variable 'changed'
X-Git-Url: http://git.cdn.openwrt.org/?a=commitdiff_plain;h=5e280420916f9483ce7b483ccc378f3c7b5929ab;p=openwrt%2Fstaging%2Fblogic.git

cfg80211: remove redundant zero check on variable 'changed'

The zero check on variable changed is redundant as it must be
between 1 and 3 at the end of the proceeding if statement block.
Remove the redundant check.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
---

diff --git a/net/wireless/wext-compat.c b/net/wireless/wext-compat.c
index d522787c7354..46e4d69db845 100644
--- a/net/wireless/wext-compat.c
+++ b/net/wireless/wext-compat.c
@@ -353,9 +353,6 @@ static int cfg80211_wext_siwretry(struct net_device *dev,
 		changed |= WIPHY_PARAM_RETRY_SHORT;
 	}
 
-	if (!changed)
-		return 0;
-
 	err = rdev_set_wiphy_params(rdev, changed);
 	if (err) {
 		wdev->wiphy->retry_short = oshort;