cfg80211: simplify strict custom alpha2 regdomain check
authorLuis R. Rodriguez <mcgrof@do-not-panic.com>
Thu, 3 Oct 2013 01:33:10 +0000 (18:33 -0700)
committerJohannes Berg <johannes.berg@intel.com>
Fri, 11 Oct 2013 13:31:43 +0000 (15:31 +0200)
This makes it easier to read.

Cc: smihir@qti.qualcomm.com
Cc: tushnimb@qca.qualcomm.com
Signed-off-by: Luis R. Rodriguez <mcgrof@do-not-panic.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/wireless/reg.c

index 37c2a63d0697c10c43096cb6f7daf7e4d1e786f7..edb2ba4e2a18f5f080cf098d82ca1e7f25b9d6ba 100644 (file)
@@ -972,6 +972,13 @@ static bool reg_dev_ignore_cell_hint(struct wiphy *wiphy)
 }
 #endif
 
+static bool wiphy_strict_alpha2_regd(struct wiphy *wiphy)
+{
+       if (wiphy->flags & WIPHY_FLAG_STRICT_REGULATORY &&
+           !(wiphy->flags & WIPHY_FLAG_CUSTOM_REGULATORY))
+               return true;
+       return false;
+}
 
 static bool ignore_reg_update(struct wiphy *wiphy,
                              enum nl80211_reg_initiator initiator)
@@ -995,9 +1002,8 @@ static bool ignore_reg_update(struct wiphy *wiphy,
         * wiphy->regd will be set once the device has its own
         * desired regulatory domain set
         */
-       if (wiphy->flags & WIPHY_FLAG_STRICT_REGULATORY && !wiphy->regd &&
+       if (wiphy_strict_alpha2_regd(wiphy) && !wiphy->regd &&
            initiator != NL80211_REGDOM_SET_BY_COUNTRY_IE &&
-           !(wiphy->flags & WIPHY_FLAG_CUSTOM_REGULATORY) &&
            !is_world_regdom(lr->alpha2)) {
                REG_DBG_PRINT("Ignoring regulatory request %s since the driver requires its own regulatory domain to be set first\n",
                              reg_initiator_name(initiator));