On 26/02/2021 22:16, Jouni Malinen wrote: > On Thu, Feb 25, 2021 at 04:06:17PM +0200, Orr Mazor wrote: >> Usefull in case we want to specify the secondary channel >> or in case we want to be on 20mghz with 40mghz capabilities. >> >> It is possible in chan_switch to set secondary channel, >> however it was not possible to do so when hostapd start. >> This commit adds this option. > > Can you please clarify how this proposed new parameter differs from the > existing ht_capab=[HT40+], ht_capab=[HT40-], and ht_capab=[HT40+][HT40-] > options? As far as I understand, the goal is to be able to specify the starting secondary channel in case ht_capab=[HT40+][HT40-] just like it's possible to specify it in the chan_switch command with the sec_channel_offset parameter. However, I guess that's just a misunderstanding of what ht_capab is doing in that case. AFAIU, ht_capab=[HT40+] gives the HT40 capability and sets the starting secondary channel to +1, but subsequent channel switches can still set it to -1. So I think this patch is indeed not needed, and instead it's just a matter of setting either [HT40+] or [HT40-] in the configuration file, and not both. Of course this way of specifying the secondary channel is fairly warty, but it is what it is :-) Regards, Arnout > > Please also note that the IEEE 802.11 standard has mandatory > requirements for 40 MHz co-existence on when the primary and secondary > channels have to be switched and I'm unlikely to accept changes > that would allow the user to bypass those requirements. > _______________________________________________ Hostap mailing list Hostap@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/hostap
