diff options
author | John W. Linville <linville@tuxdriver.com> | 2010-07-20 21:12:04 +0200 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2010-07-20 22:49:41 +0200 |
commit | 34782e9e1e58810183f2c1201124e45314b37130 (patch) | |
tree | 5958f9c467db47a0790d680d597f47866422ce5a /net/wireless/reg.c | |
parent | rt2x00: correct sparse warning in rt2x00debug.c (diff) | |
download | linux-34782e9e1e58810183f2c1201124e45314b37130.tar.xz linux-34782e9e1e58810183f2c1201124e45314b37130.zip |
wireless: remove unnecessary reg_same_country_ie_hint
"Might as well remove reg_same_country_ie_hint() completely since we
already dealt with suspend/resume through the regulatory hint
disconnect." -- Luis
Reported-by: Luis R. Rodriguez <mcgrof@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net/wireless/reg.c')
-rw-r--r-- | net/wireless/reg.c | 45 |
1 files changed, 0 insertions, 45 deletions
diff --git a/net/wireless/reg.c b/net/wireless/reg.c index 678d0bd433f0..48baf28cc4b6 100644 --- a/net/wireless/reg.c +++ b/net/wireless/reg.c @@ -1483,39 +1483,6 @@ int regulatory_hint(struct wiphy *wiphy, const char *alpha2) } EXPORT_SYMBOL(regulatory_hint); -/* Caller must hold reg_mutex */ -static bool reg_same_country_ie_hint(struct wiphy *wiphy, - char *alpha2, enum environment_cap env) -{ - struct wiphy *request_wiphy; - - assert_reg_lock(); - - if (unlikely(last_request->initiator != - NL80211_REGDOM_SET_BY_COUNTRY_IE)) - return false; - - request_wiphy = wiphy_idx_to_wiphy(last_request->wiphy_idx); - - if (!request_wiphy) - return false; - - if (likely(request_wiphy != wiphy)) - return (last_request->alpha2[0] == alpha2[0] && - last_request->alpha2[1] == alpha2[1] && - last_request->country_ie_env == env); - /* - * We should not have let these through at this point, they - * should have been picked up earlier by the first alpha2 check - * on the device - */ - if (WARN_ON((last_request->alpha2[0] == alpha2[0] && - last_request->alpha2[1] == alpha2[1] && - last_request->country_ie_env == env ))) - return true; - return false; -} - /* * We hold wdev_lock() here so we cannot hold cfg80211_mutex() and * therefore cannot iterate over the rdev list here. @@ -1560,18 +1527,6 @@ void regulatory_hint_11d(struct wiphy *wiphy, wiphy_idx_valid(last_request->wiphy_idx))) goto out; - /* - * This will not happen right now but we leave it here for the - * the future when we want to add suspend/resume support and having - * the user move to another country after doing so, or having the user - * move to another AP. Right now we just trust the first AP. - * - * If we hit this before we add this support we want to be informed of - * it as it would indicate a mistake in the current design - */ - if (WARN_ON(reg_same_country_ie_hint(wiphy, alpha2, env))) - goto free_rd_out; - request = kzalloc(sizeof(struct regulatory_request), GFP_KERNEL); if (!request) goto free_rd_out; |