(changing subject for better visibility and trimming Cc) Rafał Miłecki <rafal@xxxxxxxxxx> writes: > On 2018-11-09 15:05, Kalle Valo wrote: >> Rafał Miłecki <zajec5@xxxxxxxxx> writes: >> >>> From: Rafał Miłecki <rafal@xxxxxxxxxx> >>> >>> Driver can report IEEE80211_VHT_CAP_SUPP_CHAN_WIDTH_160MHZ so it's >>> important to provide valid & complete info about supported bands for >>> each channel. By default no support for 160 MHz should be assumed >>> unless >>> firmware reports it for a given channel later. >>> >>> This fixes info passed to the userspace. Without that change userspace >>> could try to use invalid channel and fail to start an interface. >>> >>> Signed-off-by: Rafał Miłecki <rafal@xxxxxxxxxx> >>> Cc: stable@xxxxxxxxxxxxxxx >> >> Should this be queued to 4.20? > > That's my suggestion. > > I try to mark fixes (patches for currently developed release) with an > extra FIX tag in a subject. Do you have any other method in mind that > would be preferred by you? Yes, I do see your FIX tag in patchwork: [ 31] [FIX] brcmfmac: fix reporting support for 160 MHz channels 2018-11-08 But "FIX" is a bit ambigous as not all fixes not go to wireless-drivers, they can also go to wireless-drivers-next. So I prefer using the release number (or name of the tree) like this: [PATCH 4.20] brcmfmac: fix reporting support for 160 MHz channels After seeing your question I added something about this to the wiki which hopefully helps others: https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches#tree_labels -- Kalle Valo
