diff mbox series

[1/5] supplicant: Support disabling HE for stations.

Message ID 20210306161840.16652-1-greearb@candelatech.com
State Changes Requested
Headers show
Series [1/5] supplicant: Support disabling HE for stations. | expand

Commit Message

Ben Greear March 6, 2021, 4:18 p.m. UTC 
From: Ben Greear <greearb@candelatech.com>

Send a flag to the kernel with user has specified disable_he
in supplicant config file.  Requires kernel patch to function
as expected.

Signed-off-by: Ben Greear <greearb@candelatech.com>
---
 src/drivers/driver_nl80211.c | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Ben Greear March 6, 2021, 4:21 p.m. UTC | #1 
Just FYI, the 2/5 I skipped since it needs a kernel patch that I just posted
and which may not be accepted to the kernel...

Thanks,
Ben

On 3/6/21 8:18 AM, greearb@candelatech.com wrote:
> From: Ben Greear <greearb@candelatech.com>
> 
> Send a flag to the kernel with user has specified disable_he
> in supplicant config file.  Requires kernel patch to function
> as expected.
> 
> Signed-off-by: Ben Greear <greearb@candelatech.com>
> ---
>   src/drivers/driver_nl80211.c | 8 ++++++++
>   1 file changed, 8 insertions(+)
> 
> diff --git a/src/drivers/driver_nl80211.c b/src/drivers/driver_nl80211.c
> index a9d5fe3d8..f32890d1d 100644
> --- a/src/drivers/driver_nl80211.c
> +++ b/src/drivers/driver_nl80211.c
> @@ -5913,6 +5913,14 @@ static int nl80211_ht_vht_overrides(struct nl_msg *msg,
>   	}
>   #endif /* CONFIG_VHT_OVERRIDES */
>   
> +#ifdef CONFIG_VHT_OVERRIDES
> +	if (params->disable_he) {
> +		wpa_printf(MSG_DEBUG, "  * HE disabled");
> +		if (nla_put_flag(msg, NL80211_ATTR_DISABLE_HE))
> +			return -1;
> +	}
> +#endif
> +
>   	return 0;
>   }
>   
>
Jouni Malinen March 7, 2021, 4:01 p.m. UTC | #2 
On Sat, Mar 06, 2021 at 08:18:37AM -0800, greearb@candelatech.com wrote:
> Send a flag to the kernel with user has specified disable_he
> in supplicant config file.  Requires kernel patch to function
> as expected.

What kernel patch is that referring to?

> diff --git a/src/drivers/driver_nl80211.c b/src/drivers/driver_nl80211.c
> @@ -5913,6 +5913,14 @@ static int nl80211_ht_vht_overrides(struct nl_msg *msg,
>  	}
>  #endif /* CONFIG_VHT_OVERRIDES */
>  
> +#ifdef CONFIG_VHT_OVERRIDES
> +	if (params->disable_he) {
> +		wpa_printf(MSG_DEBUG, "  * HE disabled");
> +		if (nla_put_flag(msg, NL80211_ATTR_DISABLE_HE))
> +			return -1;
> +	}
> +#endif

Shouldn't that use CONFIG_HE_OVERRIDES?
diff mbox series

Patch

diff --git a/src/drivers/driver_nl80211.c b/src/drivers/driver_nl80211.c
index a9d5fe3d8..f32890d1d 100644
--- a/src/drivers/driver_nl80211.c
+++ b/src/drivers/driver_nl80211.c
@@ -5913,6 +5913,14 @@  static int nl80211_ht_vht_overrides(struct nl_msg *msg,
 	}
 #endif /* CONFIG_VHT_OVERRIDES */
 
+#ifdef CONFIG_VHT_OVERRIDES
+	if (params->disable_he) {
+		wpa_printf(MSG_DEBUG, "  * HE disabled");
+		if (nla_put_flag(msg, NL80211_ATTR_DISABLE_HE))
+			return -1;
+	}
+#endif
+
 	return 0;
 }