[PATCH 1/2] mac80211: pass phy name to hostapd_set_bss_options

[e9hack]

Am 19.10.2020 um 21:56 schrieb David Bauer:
> hostapd_set_bss_options expects the PHY as second and the VIF as third
> argument. However, only the VIF was passed as second argument without a
> third argument at all.
> 
> This was never a problem, as both PHY and VIF were never accessed.
> However, with FTM support the PHY is needed to determine the HW support
> when configuring the BSS.
> 
> Signed-off-by: David Bauer <mail at david-bauer.net>
> ---
>  package/kernel/mac80211/files/lib/netifd/wireless/mac80211.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/package/kernel/mac80211/files/lib/netifd/wireless/mac80211.sh b/package/kernel/mac80211/files/lib/netifd/wireless/mac80211.sh
> index 42144375b4..11ea700b98 100644
> --- a/package/kernel/mac80211/files/lib/netifd/wireless/mac80211.sh
> +++ b/package/kernel/mac80211/files/lib/netifd/wireless/mac80211.sh
> @@ -372,7 +372,7 @@ mac80211_hostapd_setup_bss() {
>  	hostapd_cfg=
>  	append hostapd_cfg "$type=$ifname" "$N"
>  
> -	hostapd_set_bss_options hostapd_cfg "$vif" || return 1
> +	hostapd_set_bss_options hostapd_cfg "$phy" "$vif" || return 1
>  	json_get_vars wds wds_bridge dtim_period max_listen_int start_disabled
>  
>  	set_default wds 0
> 

Interesting, I ask for a similar patch 3 years ago and didn't get any response. For some reason, my mail isn't available in the mail archive but in patchwork:

https://patchwork.ozlabs.org/project/lede/patch/acdea515-640a-62de-afa6-45bb57654da2@gmail.com/

Regards,
Hartmut