On Tue, 2011-06-21 at 17:45 +0200, Alexander Simon wrote:
> +struct cfg80211_bss *cfg80211_get_bss_ht(struct wiphy *wiphy,
> + struct ieee80211_channel *channel,
> + const u8 *bssid,
> + const u8 *ssid, size_t ssid_len,
> + u16 capa_mask, u16 capa_val,
> + enum nl80211_channel_type channel_type);
indentation confusion?
Also, this definitely doesn't belong into a mac80211 patch. Please split
into mac80211/cfg80211.
> --- a/net/wireless/scan.c 2011-06-01 21:04:32.000000000 +0200
> +++ b/net/wireless/scan.c 2011-06-21 13:21:38.000000000 +0200
> @@ -343,6 +343,18 @@ struct cfg80211_bss *cfg80211_get_bss(st
> const u8 *ssid, size_t ssid_len,
> u16 capa_mask, u16 capa_val)
> {
> + return cfg80211_get_bss_ht(wiphy, channel, bssid, ssid, ssid_len,
> + capa_mask, capa_val, NL80211_CHAN_NO_HT);
> +}
> +EXPORT_SYMBOL(cfg80211_get_bss);
This is misleading/confusing. You're passing NO_HT in order to ignore
the channel bandwidth, but you make it seem like it's a filter. Also,
you didn't update _any_ documentation at all!
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
