[PATCH] P2P: Fix wpa_supplicant build when CONFIG_P2P=n

Jouni Malinen j at w1.fi
Wed Nov 3 04:21:46 PDT 2021


On Sat, Sep 04, 2021 at 02:01:32PM +0100, Alex Kiernan wrote:
> diff --git a/wpa_supplicant/ap.c b/wpa_supplicant/ap.c
> @@ -239,6 +239,7 @@ static void wpas_conf_ap_he_6ghz(struct wpa_supplicant *wpa_s,
> +#ifdef CONFIG_P2P
>  	if (is_chanwidth_40_80 && ssid->p2p_group &&
>  	    get_max_oper_chwidth_6ghz(ssid->max_oper_chwidth) >= 40) {
>  		conf->secondary_channel =
> @@ -254,6 +255,7 @@ static void wpas_conf_ap_he_6ghz(struct wpa_supplicant *wpa_s,
>  	if ((is_chanwidth_40_80 || is_chanwidth_160) && ssid->p2p_group &&
>  	    get_max_oper_chwidth_6ghz(ssid->max_oper_chwidth) >= 80)
>  		wpas_conf_ap_vht(wpa_s, ssid, conf, mode);
> +#endif
>  }

This would result in a number of compiler warnings due to unused
variables within the function and an unused static function for the case
where CONFIG_P2P is not defined. I'll apply a more generic way of making
this full function conditional based on this patch:
http://patchwork.ozlabs.org/project/hostap/patch/20211031230337.23390-1-masashi.honma@gmail.com/
 
-- 
Jouni Malinen                                            PGP id EFC895FA



More information about the Hostap mailing list