On Fri, 2010-12-24 at 16:44 +0900, Bruno Randolf 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)) > + if (freq == ieee80211_channel_to_frequency(12, IEEE80211_BAND_2GHZ) || > + freq == ieee80211_channel_to_frequency(13, IEEE80211_BAND_2GHZ) || > + freq == ieee80211_channel_to_frequency(14, IEEE80211_BAND_2GHZ)) > return true; This seems strange ... why not just hardcode the values?? > if (freq->e == 0) { > + int band = IEEE80211_BAND_2GHZ; enum? johannes -- 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