cfg80211: remove redundant zero check on variable 'changed'
diff mbox series

Message ID 20190327183545.20631-1-colin.king@canonical.com
State Accepted
Delegated to: Johannes Berg
Headers show
Series
  • cfg80211: remove redundant zero check on variable 'changed'
Related show

Commit Message

Colin King March 27, 2019, 6:35 p.m. UTC
From: Colin Ian King <colin.king@canonical.com>

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>
---
 net/wireless/wext-compat.c | 3 ---
 1 file changed, 3 deletions(-)

Patch
diff mbox series

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;