2023-06-15 12:19:45

by Ping-Ke Shih

[permalink] [raw]
Subject: [PATCH] wifi: rtw88: Fix action frame transmission fail before association

From: Po-Hao Huang <[email protected]>

For combo chips, antennas were controlled by bluetooth only during
power on. If WiFi wish to do transmission, notification to the coexistence
module are required. Previously we only do this before authentication.
To allow transmission before auth, such as management TX, now we start
the initiation of coexistence earlier so antennas are shared between
WiFi and bluetooth after set_channel(), and frames could then be sent.

Signed-off-by: Po-Hao Huang <[email protected]>
Signed-off-by: Ping-Ke Shih <[email protected]>
---
drivers/net/wireless/realtek/rtw88/ps.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/net/wireless/realtek/rtw88/ps.c b/drivers/net/wireless/realtek/rtw88/ps.c
index 53933fb38a330..43e80a3a8136d 100644
--- a/drivers/net/wireless/realtek/rtw88/ps.c
+++ b/drivers/net/wireless/realtek/rtw88/ps.c
@@ -18,6 +18,7 @@ static int rtw_ips_pwr_up(struct rtw_dev *rtwdev)
if (ret)
rtw_err(rtwdev, "leave idle state failed\n");

+ rtw_coex_ips_notify(rtwdev, COEX_IPS_LEAVE);
rtw_set_channel(rtwdev);

return ret;
@@ -63,8 +64,6 @@ int rtw_leave_ips(struct rtw_dev *rtwdev)

rtw_iterate_vifs(rtwdev, rtw_restore_port_cfg_iter, rtwdev);

- rtw_coex_ips_notify(rtwdev, COEX_IPS_LEAVE);
-
return 0;
}

--
2.25.1



2023-06-21 09:56:36

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH] wifi: rtw88: Fix action frame transmission fail before association

Ping-Ke Shih <[email protected]> wrote:

> From: Po-Hao Huang <[email protected]>
>
> For combo chips, antennas were controlled by bluetooth only during
> power on. If WiFi wish to do transmission, notification to the coexistence
> module are required. Previously we only do this before authentication.
> To allow transmission before auth, such as management TX, now we start
> the initiation of coexistence earlier so antennas are shared between
> WiFi and bluetooth after set_channel(), and frames could then be sent.
>
> Signed-off-by: Po-Hao Huang <[email protected]>
> Signed-off-by: Ping-Ke Shih <[email protected]>

Patch applied to wireless-next.git, thanks.

2ce9a91fe8bf wifi: rtw88: Fix action frame transmission fail before association

--
https://patchwork.kernel.org/project/linux-wireless/patch/[email protected]/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches