On 08/11/2012 02:25 AM, Luis R. Rodriguez wrote: > +static void reg_unii_5ghz_beacon_hint_unii(struct wiphy *wiphy, > + struct ieee80211_channel *chan, > + enum reg_5ghz_unii_band unii_band) > +{ > + struct ieee80211_supported_band *sband; > + unsigned int i; > + > + assert_cfg80211_lock(); > + > + sband = wiphy->bands[chan->band]; > + > + /* > + * Only process UNII 1 hints for UNII 1 channels > + * Only process UNII 3 hints for UNII 3 channels > + */ > + for (i = 0; i < sband->n_channels; i++) { > + struct ieee80211_channel *c = &sband->channels[i]; > + > + switch (unii_band) { > + case REG_5GHZ_UNII_1: > + if (!reg_is_5ghz_unii_1(c)) > + continue; > + break; > + case REG_5GHZ_UNII_3: > + if (!reg_is_5ghz_unii_3(c)) > + continue; > + break; > + default: > + WARN_ONCE(1, "unexpected regulatory unii hint\n"); > + continue; > + } > + > + reg_chan_process_beacon_hint(wiphy, chan); Not sure, but reading the description I think variable 'c' should be passed here instead of 'chan'. > + } Gr. AvS -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html