On Fri, 2022-12-02 at 10:35 -0800, Muna Sinada wrote: > > + * @ru_punct_bitmap: RU puncturing bitmap. Each bit represents a 20 MHz channel > + * with lowest bit corresponding to the lowest frequency. Bit set to 1 > + * indicates that the channel is punctured, otherwise the channel is active > + * @ru_punct_bitmap_supp_he: Indicates whether RU puncturing bitmap validation > + * should include OFDMA bitmaps. Why _supp_he and not _supp_ofdma? Is it restricted to HE? What about EHT? I'm also not really sure what that kernel-doc means for the _supp_he part? Also this patch is basically empty - please squash appropriately. johannes