On Mon, Mar 04, 2013 at 04:50:28PM +0100, Johannes Berg wrote: > On Wed, 2013-02-27 at 14:38 +0100, Karl Beldan wrote: > > > > mask = txrc->rate_idx_mask; > > > - memcpy(mcs_mask, txrc->rate_idx_mcs_mask, sizeof(mcs_mask)); > > > - if (mask != (1 << txrc->sband->n_bitrates) - 1) { > > > + if (mask != (1 << txrc->sband->n_bitrates) - 1 || > > > + memcmp(txrc->sband->ht_cap.mcs.rx_mask, txrc->rate_idx_mcs_mask, > > > + sizeof(txrc->rate_idx_mcs_mask))) { > > > + memcpy(mcs_mask, txrc->rate_idx_mcs_mask, sizeof(mcs_mask)); > > > + > > > if (sta) { > > > > > > > I don't know why I refrained myself from getting rid of the copies of > > the masks altogether. > > Are you going to resend? > Just did. -- 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