>  
> +#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

Reply via email to