cfg80211: remove redundant zero check on variable 'changed'
authorColin Ian King <colin.king@canonical.com>
Wed, 27 Mar 2019 18:35:45 +0000 (18:35 +0000)
committerJohannes Berg <johannes.berg@intel.com>
Fri, 26 Apr 2019 11:02:11 +0000 (13:02 +0200)
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>
net/wireless/wext-compat.c

index d522787c735409adca94d315da27133f50a9dd37..46e4d69db8453c35ec1d3895dfc52bb1baa08c2c 100644 (file)
@@ -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;