On Wed, 2020-05-27 at 16:32 -0700, Rajkumar Manoharan wrote: > > > However, looking at D6.0, maybe we do need some checks of the HE > > capability? > > > > > + if (!(he_cap->he_cap_elem.phy_cap_info[0] & > > > + IEEE80211_HE_PHY_CAP0_CHANNEL_WIDTH_SET_40MHZ_80MHZ_IN_5G)) > > > + return false; > > > > Looks like even D6.0 still changed something in this area... > > > > Evidently our patch just assumed that in 6 GHz all of this is > > supported, > > but the spec doesn't support that theory :-) > > > IIUC the same bits are applicable for both 5 GHz & 6 GHz. I understand > the macro doesn't capture both. Yeah, I think you're right. I looked at D6.0 (though there seems to be D6.1?) but I couldn't quite > > Can you respin this with D6.0 taken into account? > > > Let me check again and respin after your series. Does it sound good? Ok. I'll include our more limited code in the series for now then, and we can make changes to that after we're on the same page. Thanks, johannes
