On Tue, 2020-03-31 at 23:21 -0700, Thomas Pedersen wrote: > RX status needs a KHz component, so add freq_offset. We > can make ampdu_reference u16 since it is probably > sufficient to be able to distinguish 64k different > A-MPDUs. Is that necessary? Reads like we have 2 bytes free there? And we only need 13 bits for the frequency (up to 8192 MHz, 60 GHz isn't supported), so we could take out a few fractional ones for the S1G part? Dunno, I mean, we probably also don't need the A-MPDU reference, even radiotap doesn't make much representation on this, but it sort of implies that it should be unique in a capture file, for which 16 bits wouldn't be enough? (should probably clarify that though and say that you should look only "close by" or something?) johannes
