On Tue, 2011-11-08 at 13:06 -0800, Ben Greear wrote: > > About APs: that can't be right, there certainly will be 1x1 APs. > > Section 20.1.1 again: > > "An HT non-AP STA shall support all equal modulation (EQM) rates for one spatial stream (MCSs 0 through > 7) using 20 MHz channel width. An HT AP shall support all EQM rates for one and two spatial streams > (MCSs 0 through 15) using 20 MHz channel width." > > Again, I think the requirement is lame, and maybe everyone will just ignore it, > but it is there... Funny. Yeah I'm pretty sure everybody ignores that. Of course maybe you can't get a 1x1 AP certified as an AP, but surely that'll happen with a P2P GO all the time. > > start = allow_single_stream_mask ? 0 : 1; > > > > for (i = start; i< IEEE80211_HT_MCS_MASK_LEN; i++) { > > u8 val = smask[i]& scaps[i]; > > val |= ht_cap->mcs.rx_mask[i]& ~smask[i]; > > ht_cap->mcs.rx_mask[i] val; > > } > > > > or so. right? Much simpler? > > Depends on whether we want to honour the AP part of 20.1.1. > Since we don't support AP mode anyway right now, > I'm fine with your suggestion. Let me know if you want > me to proceed with your suggested changes. I don't want to honour it. In fact, the more I think about it the less I want to honour the 0-7 part here, since that just made the code more complex and almost nobody will be using this override anyway. 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