On 2013-04-19 4:06 PM, Karl Beldan wrote: > On Fri, Apr 19, 2013 at 03:01:15PM +0200, Felix Fietkau wrote: >> + if (!(rates[0].flags & IEEE80211_TX_RC_MCS)) { >> + s8 baserate = 0; >> + >> + rate = &sband->bitrates[rates[0].idx]; >> + >> + for (i = 0; i < sband->n_bitrates; i++) { >> + /* must be a basic rate */ >> + if (!(vif->bss_conf.basic_rates & BIT(i))) >> + continue; > [...] >> - if (!(info->control.rates[0].flags & IEEE80211_TX_RC_MCS)) { >> - u32 basic_rates = tx->sdata->vif.bss_conf.basic_rates; >> - s8 baserate = basic_rates ? ffs(basic_rates - 1) : 0; >> - >> - rate = &sband->bitrates[info->control.rates[0].idx]; >> - >> - for (i = 0; i < sband->n_bitrates; i++) { >> - /* must be a basic rate */ >> - if (!(basic_rates & BIT(i))) >> - continue; > > The moved code doesn't match, missing 5253ffb. Oh, I must have overlooked it during the merge. Will send v4, thanks. - Felix -- 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