Commit 69b1572b authored by Luis R. Rodriguez's avatar Luis R. Rodriguez Committed by John W. Linville

cfg80211: rename regdom_changed to regdom_changes() and use it

Signed-off-by: default avatarLuis R. Rodriguez <lrodriguez@atheros.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent fff32c04
...@@ -291,7 +291,7 @@ static bool alpha2_equal(const char *alpha2_x, const char *alpha2_y) ...@@ -291,7 +291,7 @@ static bool alpha2_equal(const char *alpha2_x, const char *alpha2_y)
return false; return false;
} }
static bool regdom_changed(const char *alpha2) static bool regdom_changes(const char *alpha2)
{ {
assert_cfg80211_lock(); assert_cfg80211_lock();
...@@ -1134,8 +1134,7 @@ static int ignore_request(struct wiphy *wiphy, enum reg_set_by set_by, ...@@ -1134,8 +1134,7 @@ static int ignore_request(struct wiphy *wiphy, enum reg_set_by set_by,
* intersect them, but that seems unlikely * intersect them, but that seems unlikely
* to be correct. Reject second one for now. * to be correct. Reject second one for now.
*/ */
if (!alpha2_equal(alpha2, if (regdom_changes(alpha2))
cfg80211_regdomain->alpha2))
return -EOPNOTSUPP; return -EOPNOTSUPP;
return -EALREADY; return -EALREADY;
} }
...@@ -1143,8 +1142,7 @@ static int ignore_request(struct wiphy *wiphy, enum reg_set_by set_by, ...@@ -1143,8 +1142,7 @@ static int ignore_request(struct wiphy *wiphy, enum reg_set_by set_by,
* Two consecutive Country IE hints on the same wiphy. * Two consecutive Country IE hints on the same wiphy.
* This should be picked up early by the driver/stack * This should be picked up early by the driver/stack
*/ */
if (WARN_ON(!alpha2_equal(cfg80211_regdomain->alpha2, if (WARN_ON(regdom_changes(alpha2)))
alpha2)))
return 0; return 0;
return -EALREADY; return -EALREADY;
} }
...@@ -1153,7 +1151,7 @@ static int ignore_request(struct wiphy *wiphy, enum reg_set_by set_by, ...@@ -1153,7 +1151,7 @@ static int ignore_request(struct wiphy *wiphy, enum reg_set_by set_by,
if (last_request->initiator == REGDOM_SET_BY_CORE) { if (last_request->initiator == REGDOM_SET_BY_CORE) {
if (is_old_static_regdom(cfg80211_regdomain)) if (is_old_static_regdom(cfg80211_regdomain))
return 0; return 0;
if (!alpha2_equal(cfg80211_regdomain->alpha2, alpha2)) if (regdom_changes(alpha2))
return 0; return 0;
return -EALREADY; return -EALREADY;
} }
...@@ -1164,7 +1162,7 @@ static int ignore_request(struct wiphy *wiphy, enum reg_set_by set_by, ...@@ -1164,7 +1162,7 @@ static int ignore_request(struct wiphy *wiphy, enum reg_set_by set_by,
* loaded card also agrees on the regulatory domain. * loaded card also agrees on the regulatory domain.
*/ */
if (last_request->initiator == REGDOM_SET_BY_DRIVER && if (last_request->initiator == REGDOM_SET_BY_DRIVER &&
alpha2_equal(cfg80211_regdomain->alpha2, alpha2)) !regdom_changes(alpha2))
return -EALREADY; return -EALREADY;
return REG_INTERSECT; return REG_INTERSECT;
...@@ -1185,13 +1183,12 @@ static int ignore_request(struct wiphy *wiphy, enum reg_set_by set_by, ...@@ -1185,13 +1183,12 @@ static int ignore_request(struct wiphy *wiphy, enum reg_set_by set_by,
if (last_request->initiator == REGDOM_SET_BY_CORE || if (last_request->initiator == REGDOM_SET_BY_CORE ||
last_request->initiator == REGDOM_SET_BY_DRIVER || last_request->initiator == REGDOM_SET_BY_DRIVER ||
last_request->initiator == REGDOM_SET_BY_USER) { last_request->initiator == REGDOM_SET_BY_USER) {
if (!alpha2_equal(last_request->alpha2, if (regdom_changes(last_request->alpha2))
cfg80211_regdomain->alpha2))
return -EAGAIN; return -EAGAIN;
} }
if (!is_old_static_regdom(cfg80211_regdomain) && if (!is_old_static_regdom(cfg80211_regdomain) &&
alpha2_equal(cfg80211_regdomain->alpha2, alpha2)) !regdom_changes(alpha2))
return -EALREADY; return -EALREADY;
return 0; return 0;
...@@ -1720,7 +1717,7 @@ static int __set_regdom(const struct ieee80211_regdomain *rd) ...@@ -1720,7 +1717,7 @@ static int __set_regdom(const struct ieee80211_regdomain *rd)
* checking if the alpha2 changes if CRDA was already called * checking if the alpha2 changes if CRDA was already called
*/ */
if (!is_old_static_regdom(cfg80211_regdomain) && if (!is_old_static_regdom(cfg80211_regdomain) &&
!regdom_changed(rd->alpha2)) !regdom_changes(rd->alpha2))
return -EINVAL; return -EINVAL;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment