Kalle Valo <kvalo@xxxxxxxxxxxxxx> writes: > Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx> wrote: > >> Adds HW_VALUE_CHANNEL(hw_value) an access macro that will be used to >> extract the channel number from struct ieee80211_channel->hw_value in >> preparation for also storing PHY settings for 802.11ac in the upper bits of >> hw_value. >> >> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx> >> Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxxxxxx> > > 5 patches applied to ath-next branch of ath.git, thanks. > > 59b5c8447c14 wcn36xx: Add accessor macro HW_VALUE_CHANNEL for hardware channels > f779a92f7b69 wcn36xx: Use HW_VALUE_CHANNEL macro to get channel number > 235b9ae0265b wcn36xx: Add accessor macro HW_VALUE_PHY for PHY settings > fc4d4008f810 wcn36xx: Encode PHY mode for 80MHz channel in hw_value > e042bc19aace wcn36xx: Set PHY into correct mode for 80MHz channel width Just as a tip for the future, I think this patchset could have been easily just one patch. Also I have noticed the same with other patchsets from you. Splitting the patches to one function per patch is too much, the recommend rule is "one logical change per patch". -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
