> -----Original Message----- > From: Kalle Valo <kvalo@xxxxxxxxxx> > Sent: Tuesday, October 10, 2023 2:20 PM > To: Ping-Ke Shih <pkshih@xxxxxxxxxxx> > Cc: johannes@xxxxxxxxxxxxxxxx; linux-wireless@xxxxxxxxxxxxxxx > Subject: Re: [PATCH v2 5/6] wifi: radiotap: add bandwidth definition of EHT U-SIG > > Ping-Ke Shih <pkshih@xxxxxxxxxxx> writes: > > > Define EHT U-SIG bandwidth used by radiotap according to Table 36-28 > > "U-SIG field of an EHT MU PPDU" in 802.11be (D3.0). > > > > Signed-off-by: Ping-Ke Shih <pkshih@xxxxxxxxxxx> > > --- > > v2: > > - correct commit message about the source of bandwidth definition > > of EHT U-SIG suggested by Johannes > > --- > > include/net/ieee80211_radiotap.h | 6 ++++++ > > 1 file changed, 6 insertions(+) > > This is nitpicking but in general it's good to have the patches touching > files outside of drivers/net/wireless in the beginning of the patchset > to make them more visible. No need to resend because of this, just a > small tip. > Got it. I will pay attention to it next time. Ping-Ke