regulatory: remove handling of channel bandwidth
authorJohannes Berg <johannes.berg@intel.com>
Tue, 4 Dec 2012 14:07:34 +0000 (15:07 +0100)
committerJohannes Berg <johannes.berg@intel.com>
Thu, 3 Jan 2013 12:01:28 +0000 (13:01 +0100)
The channel bandwidth handling isn't really quite right,
it assumes that a 40 MHz channel is really two 20 MHz
channels, which isn't strictly true. This is the way the
regulatory database handling is defined right now though
so remove the logic to handle other channel widths.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
drivers/net/wireless/ath/regd.c
drivers/net/wireless/brcm80211/brcmsmac/channel.c
drivers/net/wireless/rtlwifi/regd.c
include/net/cfg80211.h
net/wireless/reg.c

index d81698015bf75897ae9d58fcf80e4ef6359e4be9..8ae58c404c673f1d25ff7b39db5c9fd11b173bd2 100644 (file)
@@ -195,7 +195,6 @@ ath_reg_apply_beaconing_flags(struct wiphy *wiphy,
        const struct ieee80211_reg_rule *reg_rule;
        struct ieee80211_channel *ch;
        unsigned int i;
-       u32 bandwidth = 0;
        int r;
 
        for (band = 0; band < IEEE80211_NUM_BANDS; band++) {
@@ -216,7 +215,6 @@ ath_reg_apply_beaconing_flags(struct wiphy *wiphy,
                        if (initiator == NL80211_REGDOM_SET_BY_COUNTRY_IE) {
                                r = freq_reg_info(wiphy,
                                                  ch->center_freq,
-                                                 bandwidth,
                                                  &reg_rule);
                                if (r)
                                        continue;
@@ -254,7 +252,6 @@ ath_reg_apply_active_scan_flags(struct wiphy *wiphy,
        struct ieee80211_supported_band *sband;
        struct ieee80211_channel *ch;
        const struct ieee80211_reg_rule *reg_rule;
-       u32 bandwidth = 0;
        int r;
 
        sband = wiphy->bands[IEEE80211_BAND_2GHZ];
@@ -283,7 +280,7 @@ ath_reg_apply_active_scan_flags(struct wiphy *wiphy,
         */
 
        ch = &sband->channels[11]; /* CH 12 */
-       r = freq_reg_info(wiphy, ch->center_freq, bandwidth, &reg_rule);
+       r = freq_reg_info(wiphy, ch->center_freq, &reg_rule);
        if (!r) {
                if (!(reg_rule->flags & NL80211_RRF_PASSIVE_SCAN))
                        if (ch->flags & IEEE80211_CHAN_PASSIVE_SCAN)
@@ -291,7 +288,7 @@ ath_reg_apply_active_scan_flags(struct wiphy *wiphy,
        }
 
        ch = &sband->channels[12]; /* CH 13 */
-       r = freq_reg_info(wiphy, ch->center_freq, bandwidth, &reg_rule);
+       r = freq_reg_info(wiphy, ch->center_freq, &reg_rule);
        if (!r) {
                if (!(reg_rule->flags & NL80211_RRF_PASSIVE_SCAN))
                        if (ch->flags & IEEE80211_CHAN_PASSIVE_SCAN)
index a90b72202ec5a298b9cf6483cd5380ae5ef74ef0..30272519d795452655618657ae5578415b728017 100644 (file)
@@ -686,7 +686,7 @@ brcms_reg_apply_beaconing_flags(struct wiphy *wiphy,
 
                        if (initiator == NL80211_REGDOM_SET_BY_COUNTRY_IE) {
                                ret = freq_reg_info(wiphy, ch->center_freq,
-                                                   0, &rule);
+                                                   &rule);
                                if (ret)
                                        continue;
 
index c1608cddc5299e5150a323d0a3efb7a2f0b63de6..be55dc9167f070813e821365303b63558ac74dcd 100644 (file)
@@ -158,7 +158,6 @@ static void _rtl_reg_apply_beaconing_flags(struct wiphy *wiphy,
        const struct ieee80211_reg_rule *reg_rule;
        struct ieee80211_channel *ch;
        unsigned int i;
-       u32 bandwidth = 0;
        int r;
 
        for (band = 0; band < IEEE80211_NUM_BANDS; band++) {
@@ -174,8 +173,7 @@ static void _rtl_reg_apply_beaconing_flags(struct wiphy *wiphy,
                            (ch->flags & IEEE80211_CHAN_RADAR))
                                continue;
                        if (initiator == NL80211_REGDOM_SET_BY_COUNTRY_IE) {
-                               r = freq_reg_info(wiphy, ch->center_freq,
-                                                 bandwidth, &reg_rule);
+                               r = freq_reg_info(wiphy, ch->center_freq, &reg_rule);
                                if (r)
                                        continue;
 
@@ -211,7 +209,6 @@ static void _rtl_reg_apply_active_scan_flags(struct wiphy *wiphy,
        struct ieee80211_supported_band *sband;
        struct ieee80211_channel *ch;
        const struct ieee80211_reg_rule *reg_rule;
-       u32 bandwidth = 0;
        int r;
 
        if (!wiphy->bands[IEEE80211_BAND_2GHZ])
@@ -240,7 +237,7 @@ static void _rtl_reg_apply_active_scan_flags(struct wiphy *wiphy,
         */
 
        ch = &sband->channels[11];      /* CH 12 */
-       r = freq_reg_info(wiphy, ch->center_freq, bandwidth, &reg_rule);
+       r = freq_reg_info(wiphy, ch->center_freq, &reg_rule);
        if (!r) {
                if (!(reg_rule->flags & NL80211_RRF_PASSIVE_SCAN))
                        if (ch->flags & IEEE80211_CHAN_PASSIVE_SCAN)
@@ -248,7 +245,7 @@ static void _rtl_reg_apply_active_scan_flags(struct wiphy *wiphy,
        }
 
        ch = &sband->channels[12];      /* CH 13 */
-       r = freq_reg_info(wiphy, ch->center_freq, bandwidth, &reg_rule);
+       r = freq_reg_info(wiphy, ch->center_freq, &reg_rule);
        if (!r) {
                if (!(reg_rule->flags & NL80211_RRF_PASSIVE_SCAN))
                        if (ch->flags & IEEE80211_CHAN_PASSIVE_SCAN)
index 8e6a6b73b9c9ecc3f8f221c4ff32a9983940dde4..c222e5fbf53aa894d9da7f47f7c7fa65059658db 100644 (file)
@@ -2938,10 +2938,6 @@ extern void wiphy_apply_custom_regulatory(
  * freq_reg_info - get regulatory information for the given frequency
  * @wiphy: the wiphy for which we want to process this rule for
  * @center_freq: Frequency in KHz for which we want regulatory information for
- * @desired_bw_khz: the desired max bandwidth you want to use per
- *     channel. Note that this is still 20 MHz if you want to use HT40
- *     as HT40 makes use of two channels for its 40 MHz width bandwidth.
- *     If set to 0 we'll assume you want the standard 20 MHz.
  * @reg_rule: the regulatory rule which we have for this frequency
  *
  * Use this function to get the regulatory rule for a specific frequency on
@@ -2956,9 +2952,7 @@ extern void wiphy_apply_custom_regulatory(
  * freq_in_rule_band() for our current definition of a band -- this is purely
  * subjective and right now its 802.11 specific.
  */
-extern int freq_reg_info(struct wiphy *wiphy,
-                        u32 center_freq,
-                        u32 desired_bw_khz,
+extern int freq_reg_info(struct wiphy *wiphy, u32 center_freq,
                         const struct ieee80211_reg_rule **reg_rule);
 
 /*
index b3f94c957d1daa654840694135daa115a9f53b6f..2e38b47939a3e763c231cc11c0ce2316c494c26b 100644 (file)
@@ -480,8 +480,7 @@ static bool is_valid_rd(const struct ieee80211_regdomain *rd)
 }
 
 static bool reg_does_bw_fit(const struct ieee80211_freq_range *freq_range,
-                           u32 center_freq_khz,
-                           u32 bw_khz)
+                           u32 center_freq_khz, u32 bw_khz)
 {
        u32 start_freq_khz, end_freq_khz;
 
@@ -682,9 +681,7 @@ static u32 map_regdom_flags(u32 rd_flags)
        return channel_flags;
 }
 
-static int freq_reg_info_regd(struct wiphy *wiphy,
-                             u32 center_freq,
-                             u32 desired_bw_khz,
+static int freq_reg_info_regd(struct wiphy *wiphy, u32 center_freq,
                              const struct ieee80211_reg_rule **reg_rule,
                              const struct ieee80211_regdomain *regd)
 {
@@ -692,9 +689,6 @@ static int freq_reg_info_regd(struct wiphy *wiphy,
        bool band_rule_found = false;
        bool bw_fits = false;
 
-       if (!desired_bw_khz)
-               desired_bw_khz = MHZ_TO_KHZ(20);
-
        if (!regd)
                return -EINVAL;
 
@@ -713,7 +707,7 @@ static int freq_reg_info_regd(struct wiphy *wiphy,
                if (!band_rule_found)
                        band_rule_found = freq_in_rule_band(fr, center_freq);
 
-               bw_fits = reg_does_bw_fit(fr, center_freq, desired_bw_khz);
+               bw_fits = reg_does_bw_fit(fr, center_freq, MHZ_TO_KHZ(20));
 
                if (band_rule_found && bw_fits) {
                        *reg_rule = rr;
@@ -727,7 +721,7 @@ static int freq_reg_info_regd(struct wiphy *wiphy,
        return -EINVAL;
 }
 
-int freq_reg_info(struct wiphy *wiphy, u32 center_freq, u32 desired_bw_khz,
+int freq_reg_info(struct wiphy *wiphy, u32 center_freq,
                  const struct ieee80211_reg_rule **reg_rule)
 {
        const struct ieee80211_regdomain *regd;
@@ -746,8 +740,7 @@ int freq_reg_info(struct wiphy *wiphy, u32 center_freq, u32 desired_bw_khz,
        else
                regd = cfg80211_regdomain;
 
-       return freq_reg_info_regd(wiphy, center_freq, desired_bw_khz,
-                                 reg_rule, regd);
+       return freq_reg_info_regd(wiphy, center_freq, reg_rule, regd);
 }
 EXPORT_SYMBOL(freq_reg_info);
 
@@ -770,7 +763,6 @@ static const char *reg_initiator_name(enum nl80211_reg_initiator initiator)
 }
 
 static void chan_reg_rule_print_dbg(struct ieee80211_channel *chan,
-                                   u32 desired_bw_khz,
                                    const struct ieee80211_reg_rule *reg_rule)
 {
        const struct ieee80211_power_rule *power_rule;
@@ -785,8 +777,8 @@ static void chan_reg_rule_print_dbg(struct ieee80211_channel *chan,
        else
                snprintf(max_antenna_gain, 32, "%d", power_rule->max_antenna_gain);
 
-       REG_DBG_PRINT("Updating information on frequency %d MHz for a %d MHz width channel with regulatory rule:\n",
-                     chan->center_freq, KHZ_TO_MHZ(desired_bw_khz));
+       REG_DBG_PRINT("Updating information on frequency %d MHz with regulatory rule:\n",
+                     chan->center_freq);
 
        REG_DBG_PRINT("%d KHz - %d KHz @ %d KHz), (%s mBi, %d mBm)\n",
                      freq_range->start_freq_khz, freq_range->end_freq_khz,
@@ -795,7 +787,6 @@ static void chan_reg_rule_print_dbg(struct ieee80211_channel *chan,
 }
 #else
 static void chan_reg_rule_print_dbg(struct ieee80211_channel *chan,
-                                   u32 desired_bw_khz,
                                    const struct ieee80211_reg_rule *reg_rule)
 {
        return;
@@ -805,11 +796,7 @@ static void chan_reg_rule_print_dbg(struct ieee80211_channel *chan,
 /*
  * Note that right now we assume the desired channel bandwidth
  * is always 20 MHz for each individual channel (HT40 uses 20 MHz
- * per channel, the primary and the extension channel). To support
- * smaller custom bandwidths such as 5 MHz or 10 MHz we'll need a
- * new ieee80211_channel.target_bw and re run the regulatory check
- * on the wiphy with the target_bw specified. Then we can simply use
- * that below for the desired_bw_khz below.
+ * per channel, the primary and the extension channel).
  */
 static void handle_channel(struct wiphy *wiphy,
                           enum nl80211_reg_initiator initiator,
@@ -817,7 +804,6 @@ static void handle_channel(struct wiphy *wiphy,
 {
        int r;
        u32 flags, bw_flags = 0;
-       u32 desired_bw_khz = MHZ_TO_KHZ(20);
        const struct ieee80211_reg_rule *reg_rule = NULL;
        const struct ieee80211_power_rule *power_rule = NULL;
        const struct ieee80211_freq_range *freq_range = NULL;
@@ -829,8 +815,7 @@ static void handle_channel(struct wiphy *wiphy,
 
        flags = chan->orig_flags;
 
-       r = freq_reg_info(wiphy, MHZ_TO_KHZ(chan->center_freq),
-                         desired_bw_khz, &reg_rule);
+       r = freq_reg_info(wiphy, MHZ_TO_KHZ(chan->center_freq), &reg_rule);
        if (r) {
                /*
                 * We will disable all channels that do not match our
@@ -851,7 +836,7 @@ static void handle_channel(struct wiphy *wiphy,
                return;
        }
 
-       chan_reg_rule_print_dbg(chan, desired_bw_khz, reg_rule);
+       chan_reg_rule_print_dbg(chan, reg_rule);
 
        power_rule = &reg_rule->power_rule;
        freq_range = &reg_rule->freq_range;
@@ -1223,23 +1208,22 @@ static void handle_channel_custom(struct wiphy *wiphy,
                                  const struct ieee80211_regdomain *regd)
 {
        int r;
-       u32 desired_bw_khz = MHZ_TO_KHZ(20);
        u32 bw_flags = 0;
        const struct ieee80211_reg_rule *reg_rule = NULL;
        const struct ieee80211_power_rule *power_rule = NULL;
        const struct ieee80211_freq_range *freq_range = NULL;
 
        r = freq_reg_info_regd(wiphy, MHZ_TO_KHZ(chan->center_freq),
-                              desired_bw_khz, &reg_rule, regd);
+                              &reg_rule, regd);
 
        if (r) {
-               REG_DBG_PRINT("Disabling freq %d MHz as custom regd has no rule that fits a %d MHz wide channel\n",
-                             chan->center_freq, KHZ_TO_MHZ(desired_bw_khz));
+               REG_DBG_PRINT("Disabling freq %d MHz as custom regd has no rule that fits it\n",
+                             chan->center_freq);
                chan->flags = IEEE80211_CHAN_DISABLED;
                return;
        }
 
-       chan_reg_rule_print_dbg(chan, desired_bw_khz, reg_rule);
+       chan_reg_rule_print_dbg(chan, reg_rule);
 
        power_rule = &reg_rule->power_rule;
        freq_range = &reg_rule->freq_range;