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? 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