> + if (!ieee80211_is_data(hdr->frame_control)) { Let's also check ieee80211_is_data_qos. > +#define WCN36XX_BD_RATE_DATA 0 Since this define is not used anywhere what about having it as a comment? -- Best regards, Eugene