On Mon, 2016-05-02 at 14:23 -0700, greearb@xxxxxxxxxxxxxxx wrote: > > + PFLAG(MAX_MPDU_LENGTH_3895, "MAX-MPDU-3895"); > + PFLAG(MAX_MPDU_LENGTH_7991, "MAX-MPDU-7991"); > + PFLAG(MAX_MPDU_LENGTH_11454, "MAX-MPDU-11454"); This doesn't work as intended, please treat this two-bit field properly. > + PFLAG(SUPP_CHAN_WIDTH_160MHZ, "160Mhz"); > + PFLAG(SUPP_CHAN_WIDTH_160_80PLUS80MHZ, "160- > 80+80Mhz"); Same here. > + PFLAG(RXSTBC_1, "RXSTBC_1"); > + PFLAG(RXSTBC_2, "RXSTBC_2"); > + PFLAG(RXSTBC_3, "RXSTBC_3"); > + PFLAG(RXSTBC_4, "RXSTBC_4"); This is clearly also wrong, as you could even see in the output you pasted into the commit log. johannes -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html