On Mon, Feb 23, 2009 at 05:58:50PM -0800, Johannes Berg wrote: > On Sat, 2009-02-21 at 00:20 -0500, Luis R. Rodriguez wrote: > > > +static bool freq_is_chan_12_13_14(u16 freq) > > +{ > > + if (freq == ieee80211_channel_to_frequency(12) || > > + freq == ieee80211_channel_to_frequency(13) || > > + freq == ieee80211_channel_to_frequency(14)) > > + return true; > > + return false; > > +} > > + > > +int regulatory_hint_found_beacon(struct wiphy *wiphy, > > + struct ieee80211_channel *beacon_chan, > > + gfp_t gfp) > > +{ > > + struct reg_beacon *reg_beacon; > > + > > + if (likely((beacon_chan->beacon_found || > > + (beacon_chan->flags & IEEE80211_CHAN_RADAR) || > > + (beacon_chan->band == IEEE80211_BAND_2GHZ && > > + !freq_is_chan_12_13_14(beacon_chan->center_freq))))) > > + return 0; > > This doesn't seem right, shouldn't it use the world regulatory domain to > check against rather than hardcoding it? I'm not sure I understand, can you elaborate on what you mean. Luis -- 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