> > +#define IEEE80211_ADDBA_EXT_ELEM_ID 0x9f That doesn't really belong here. > +#define IEEE80211_ADDBA_EXT_ELEM_ID_LEN 0x01 And that's usually not a define? > +#define IEEE80211_ADDBA_EXT_FRAG_LEVEL_MASK GENMASK(2, 1) > +#define IEEE80211_ADDBA_EXT_FRAG_LEVEL_SHIFT 1 > +#define IEEE80211_ADDBA_EXT_NO_FRAG BIT(0) > + > +struct ieee80211_addbaext { > + u8 eid; > + u8 length; > + u8 data; > +} __packed; I think we don't typically put the eid/len inside, and then we use sizeof(struct ...) for the size. johannes