aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJohannes Berg <johannes.berg@intel.com>2012-11-26 12:51:52 +0100
committerJohannes Berg <johannes.berg@intel.com>2012-11-28 13:26:23 +0100
commitd9d8b01978f6510a262ddc4da3b3694d5b149cfe (patch)
treee0e8f5e476bdbb2b40a5fbfd646deb6476369dc8
parent339afbf4819e5c7c0a0422af43b8c2eccd059abf (diff)
downloadkernel_goldelico_gta04-d9d8b01978f6510a262ddc4da3b3694d5b149cfe.zip
kernel_goldelico_gta04-d9d8b01978f6510a262ddc4da3b3694d5b149cfe.tar.gz
kernel_goldelico_gta04-d9d8b01978f6510a262ddc4da3b3694d5b149cfe.tar.bz2
nl80211: remove unnecessary checks
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>
-rw-r--r--net/wireless/nl80211.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/net/wireless/nl80211.c b/net/wireless/nl80211.c
index 7f53aaf..7cce43b 100644
--- a/net/wireless/nl80211.c
+++ b/net/wireless/nl80211.c
@@ -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;