On Fri, 2015-01-16 at 11:38 +0100, Janusz Dziedzic wrote:
> In some cases when used HT40+/HT40-, center_freq1
> was configured incorrectly. Next this couse
couse?
> @@ -407,10 +407,15 @@ static void ieee80211_sta_join_ibss(struct
> ieee80211_sub_if_data *sdata,
> switch (sdata->u.ibss.chandef.width) {
> case NL80211_CHAN_WIDTH_20_NOHT:
> case NL80211_CHAN_WIDTH_20:
> - case NL80211_CHAN_WIDTH_40:
> chan_type = cfg80211_get_chandef_type(&sdata->u.ibss.chandef);
> cfg80211_chandef_create(&chandef, cbss->channel, chan_type);
> break;
Sorry, I don't see the bug. "Sometimes"?
cfg80211_get_chandef_type() will return HT40+/- correctly according to
the chandef, and then you'll use it again to create the chandef - should be OK.
> + case NL80211_CHAN_WIDTH_40:
> + cfg80211_chandef_create(&chandef, cbss->channel,
> + NL80211_CHAN_WIDTH_20_NOHT);
> + chandef.width = sdata->u.ibss.chandef.width;
> + chandef.center_freq1 =
> sdata->u.ibss.chandef.center_freq1;
> + break;
Please don't mix chantype stuff and chandef stuff ... just create the
chandef manually if you must. I really think the code above should have
worked though, unless it's using the wrong inputs somehow.
johannes
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
