developer | e35b8e4 | 2023-10-16 11:04:00 +0800 | [diff] [blame] | 1 | From 5b750f3f78cec3b64ebb9bb3b74def424c7d57fd Mon Sep 17 00:00:00 2001 |
developer | 2a20969 | 2023-08-14 20:23:42 +0800 | [diff] [blame] | 2 | From: StanleyYP Wang <StanleyYP.Wang@mediatek.com> |
| 3 | Date: Tue, 8 Aug 2023 19:21:41 +0800 |
developer | e35b8e4 | 2023-10-16 11:04:00 +0800 | [diff] [blame] | 4 | Subject: [PATCH 33/38] hostapd: mtk: add back ht vht cap missing field before |
developer | dfb5098 | 2023-09-11 13:34:36 +0800 | [diff] [blame] | 5 | dfs channel fallback |
developer | 2a20969 | 2023-08-14 20:23:42 +0800 | [diff] [blame] | 6 | |
| 7 | hostapd_event_ch_switch would set / clear ht_capab and vht_capab, based |
| 8 | on the bandwidth of switched channel. |
| 9 | For example, vht bw 160 support field would be cleared if we switch to |
| 10 | non bw 160 channel. |
| 11 | This design works fine with NON-DFS channel switch. |
| 12 | However, for those DFS channels who require CAC, channel switch command |
| 13 | calls hostapd_switch_channel_fallback instead of hostapd_switch_channel. |
| 14 | This is simply restarting the interface not CHANNEL SWITCHING, so |
| 15 | hostapd will not receive any ch_switch event from kernel. |
| 16 | Therefore, the cleared field in vht_capab will not be set back to 1, |
| 17 | even if we channel switch to dfs channel bw 160. |
| 18 | |
| 19 | Signed-off-by: StanleyYP Wang <StanleyYP.Wang@mediatek.com> |
| 20 | --- |
| 21 | src/ap/hostapd.c | 7 +++++++ |
| 22 | 1 file changed, 7 insertions(+) |
| 23 | |
| 24 | diff --git a/src/ap/hostapd.c b/src/ap/hostapd.c |
developer | e35b8e4 | 2023-10-16 11:04:00 +0800 | [diff] [blame] | 25 | index 3e184b5..b8ae8de 100644 |
developer | 2a20969 | 2023-08-14 20:23:42 +0800 | [diff] [blame] | 26 | --- a/src/ap/hostapd.c |
| 27 | +++ b/src/ap/hostapd.c |
developer | dfb5098 | 2023-09-11 13:34:36 +0800 | [diff] [blame] | 28 | @@ -4111,6 +4111,13 @@ hostapd_switch_channel_fallback(struct hostapd_iface *iface, |
developer | 2a20969 | 2023-08-14 20:23:42 +0800 | [diff] [blame] | 29 | break; |
| 30 | } |
| 31 | |
| 32 | + if ((iface->current_mode->ht_capab & HT_CAP_INFO_SUPP_CHANNEL_WIDTH_SET) && |
| 33 | + freq_params->bandwidth > 20) |
| 34 | + iface->conf->ht_capab |= HT_CAP_INFO_SUPP_CHANNEL_WIDTH_SET; |
| 35 | + if ((iface->current_mode->vht_capab & VHT_CAP_SUPP_CHAN_WIDTH_160MHZ) && |
| 36 | + freq_params->bandwidth == 160) |
| 37 | + iface->conf->vht_capab |= VHT_CAP_SUPP_CHAN_WIDTH_160MHZ; |
| 38 | + |
| 39 | iface->freq = freq_params->freq; |
| 40 | iface->conf->channel = freq_params->channel; |
| 41 | iface->conf->secondary_channel = freq_params->sec_channel_offset; |
| 42 | -- |
| 43 | 2.18.0 |
| 44 | |