On Tue, 2011-06-21 at 17:53 +0200, Alexander Simon wrote: > + /* entering a legacy IBSS. Use given HT configuration. */ > + if (channel_type == NL80211_CHAN_NO_HT) > + channel_type = ifibss->channel_type; > + > local->oper_channel = chan; > - WARN_ON(!ieee80211_set_channel_type(local, sdata, NL80211_CHAN_NO_HT)); > + WARN_ON(!ieee80211_set_channel_type(local, sdata, channel_type)); I don't think you can now safely assume that set_channel_type() will succeed, but I didn't check the chan.c code for IBSS now. Please explain why this is OK now if you think that it is. NO_HT is always compatible, but the HT mode you used won't necessarily be. I think for fixed-channel mode this might be an issue. > + local->_oper_channel_type = > + channel_type; > + WARN_ON(!ieee80211_set_channel_type( > + local, > + sdata, > + channel_type)); Same here of course. > @@ -404,7 +482,7 @@ static void ieee80211_rx_bss_info(struct > ieee80211_sta_join_ibss(sdata, bss); > supp_rates = ieee80211_sta_get_rates(local, elems, band); > ieee80211_ibss_add_sta(sdata, mgmt->bssid, mgmt->sa, > - supp_rates, GFP_KERNEL); > + supp_rates, elems->ht_cap_elem, GFP_KERNEL); indentation? 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
