Hi Michael, > I will fix the typos and change all the “IE ” to “Element”. The TODO > for 320 MHz bandwidth is used for internal tests, I’ll remove it, too. Sounds good. > We had checked the D4.1, and unfortunately, the additional description > for WBCS Element in D3.2 9.4.2.159 is removed. In other words, D4.1 > does not change the definitions of WBCS Element and WBCS Element should > be parsed as VHT operating information regardless of the BSS being VHT, > HE, or EHT. Right, I thought there were going to be some changes here. > After parsing the WBCS Element, the STA needs to check whether the new > channel fits its capabilities according to the operating mode > (VHT/HE/EHT). > However, the existing flow only checks the VHT capability after the > WBCS Element parsing, which is incorrect when the BSS is HE/EHT or the > band is 6 GHz. > > In summary, I will refactor the WBCS Element parsing part of this > patch, along with other fixes. Sounds good. We'll have to figure this out separately, but I'm also working on something else that will certainly affect this code. I noticed very recently that the whole STA_DISABLE_* flags are very messy, and am working on some improvements along these lines. The current version looks like this, but it doesn't even compile yet: https://p.sipsolutions.net/ccb552810b020745.txt No idea if this would help/hinder you in any way, but I figured since it has some overlap I'd let you know. I don't think you need to worry about it for now, it'll take me some more time to work on it. johannes
