nl80211: remove unnecessary checks
authorJohannes Berg <johannes.berg@intel.com>
Mon, 26 Nov 2012 11:51:52 +0000 (12:51 +0100)
committerJohannes Berg <johannes.berg@intel.com>
Wed, 28 Nov 2012 12:26:23 +0000 (13:26 +0100)
The CQM TX-error rate/interval can't be less than
zero since they're unsigned values, remove checks.
Also fix indentation of the function.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/wireless/nl80211.c

index 7f53aafd47f7032fac5e09ab0883b5ba114eabc1..7cce43bc948949d273b904bf2997a98b3a5c4bd3 100644 (file)
@@ -6555,14 +6555,13 @@ nl80211_attr_cqm_policy[NL80211_ATTR_CQM_MAX + 1] __read_mostly = {
 };
 
 static int nl80211_set_cqm_txe(struct genl_info *info,
-                               u32 rate, u32 pkts, u32 intvl)
+                              u32 rate, u32 pkts, u32 intvl)
 {
        struct cfg80211_registered_device *rdev = info->user_ptr[0];
        struct wireless_dev *wdev;
        struct net_device *dev = info->user_ptr[1];
 
-       if ((rate < 0 || rate > 100) ||
-           (intvl < 0 || intvl > NL80211_CQM_TXE_MAX_INTVL))
+       if (rate > 100 || intvl > NL80211_CQM_TXE_MAX_INTVL)
                return -EINVAL;
 
        wdev = dev->ieee80211_ptr;