---
 drivers/net/wireless/realtek/rtw89/pci.c    | 17 ++++++++++++++---
 drivers/net/wireless/realtek/rtw89/pci.h    | 11 +++++++++++
 drivers/net/wireless/realtek/rtw89/pci_be.c |  2 ++
 3 files changed, 27 insertions(+), 3 deletions(-)

drivers/net/wireless/realtek/rtw89/pci.c +14 -3

index 5aef7fa37..0ac84f968 100644
@@ -2492,7 +2492,7 @@ static int rtw89_pci_dphy_delay(struct rtw89_dev *rtwdev)
 				       PCIE_DPHY_DLY_25US, PCIE_PHY_GEN1);
 }
 
-static void rtw89_pci_power_wake(struct rtw89_dev *rtwdev, bool pwr_up)
+static void rtw89_pci_power_wake_ax(struct rtw89_dev *rtwdev, bool pwr_up)
 {
 	if (pwr_up)
 		rtw89_write32_set(rtwdev, R_AX_HCI_OPT_CTRL, BIT_WAKE_CTRL);

@@ -2799,6 +2799,8 @@ static int rtw89_pci_ops_deinit(struct rtw89_dev *rtwdev)
 {
 	const struct rtw89_pci_info *info = rtwdev->pci_info;
 
+	rtw89_pci_power_wake(rtwdev, false);
+
 	if (rtwdev->chip->chip_id == RTL8852A) {
 		/* ltr sw trigger */
 		rtw89_write32_set(rtwdev, R_AX_LTR_CTRL_0, B_AX_APP_LTR_IDLE);

@@ -2841,7 +2843,7 @@ static int rtw89_pci_ops_mac_pre_init_ax(struct rtw89_dev *rtwdev)
 		return ret;
 	}
 
-	rtw89_pci_power_wake(rtwdev, true);
+	rtw89_pci_power_wake_ax(rtwdev, true);
 	rtw89_pci_autoload_hang(rtwdev);
 	rtw89_pci_l12_vmain(rtwdev);
 	rtw89_pci_gen2_force_ib(rtwdev);

@@ -2886,6 +2888,13 @@ static int rtw89_pci_ops_mac_pre_init_ax(struct rtw89_dev *rtwdev)
 	return 0;
 }
 
+static int rtw89_pci_ops_mac_pre_deinit_ax(struct rtw89_dev *rtwdev)
+{
+	rtw89_pci_power_wake_ax(rtwdev, false);
+
+	return 0;
+}
+
 int rtw89_pci_ltr_set(struct rtw89_dev *rtwdev, bool en)
 {
 	u32 val;

@@ -4264,7 +4273,7 @@ const struct rtw89_pci_gen_def rtw89_pci_gen_ax = {
 					    B_AX_RDU_INT},
 
 	.mac_pre_init = rtw89_pci_ops_mac_pre_init_ax,
-	.mac_pre_deinit = NULL,
+	.mac_pre_deinit = rtw89_pci_ops_mac_pre_deinit_ax,
 	.mac_post_init = rtw89_pci_ops_mac_post_init_ax,
 
 	.clr_idx_all = rtw89_pci_clr_idx_all_ax,

@@ -4280,6 +4289,8 @@ const struct rtw89_pci_gen_def rtw89_pci_gen_ax = {
 	.aspm_set = rtw89_pci_aspm_set_ax,
 	.clkreq_set = rtw89_pci_clkreq_set_ax,
 	.l1ss_set = rtw89_pci_l1ss_set_ax,
+
+	.power_wake = rtw89_pci_power_wake_ax,
 };
 EXPORT_SYMBOL(rtw89_pci_gen_ax);

drivers/net/wireless/realtek/rtw89/pci.h +11 -0

index 48c3ab735..0ea4dcb84 100644
@@ -1276,6 +1276,8 @@ struct rtw89_pci_gen_def {
 	void (*aspm_set)(struct rtw89_dev *rtwdev, bool enable);
 	void (*clkreq_set)(struct rtw89_dev *rtwdev, bool enable);
 	void (*l1ss_set)(struct rtw89_dev *rtwdev, bool enable);
+
+	void (*power_wake)(struct rtw89_dev *rtwdev, bool pwr_up);
 };
 
 struct rtw89_pci_info {

@@ -1766,4 +1768,13 @@ static inline int rtw89_pci_poll_txdma_ch_idle(struct rtw89_dev *rtwdev)
 
 	return gen_def->poll_txdma_ch_idle(rtwdev);
 }
+
+static inline void rtw89_pci_power_wake(struct rtw89_dev *rtwdev, bool pwr_up)
+{
+	const struct rtw89_pci_info *info = rtwdev->pci_info;
+	const struct rtw89_pci_gen_def *gen_def = info->gen_def;
+
+	gen_def->power_wake(rtwdev, pwr_up);
+}
+
 #endif

drivers/net/wireless/realtek/rtw89/pci_be.c +2 -0

index 7cc328222..2f0d9ff25 100644
@@ -614,5 +614,7 @@ const struct rtw89_pci_gen_def rtw89_pci_gen_be = {
 	.aspm_set = rtw89_pci_aspm_set_be,
 	.clkreq_set = rtw89_pci_clkreq_set_be,
 	.l1ss_set = rtw89_pci_l1ss_set_be,
+
+	.power_wake = _patch_pcie_power_wake_be,
 };
 EXPORT_SYMBOL(rtw89_pci_gen_be);

Signature · 1 line

2.48.1

Greg KH

maintainer

2025-02-11 10:33 UTC #1 lore raw

Quoted 7 lines · Zenm Chen

From: Ping-Ke Shih <[email protected]>
[ Upstream commit 9c1df813e08832c3836c254bc8a2f83ff22dbc06 ]
The PCIE wake bit is to control PCIE wake signal to host. When PCIE is going down, clear this bit to prevent waking up host unexpectedly.

Quoted 6 lines

Signed-off-by Ping-Ke Shih <[email protected]>
Link https://patch.msgid.link/[email protected] open here
Signed-off-by Zenm Chen <[email protected]>
---
Some users of RTL8852BE chip may encounter a shutdown issue [1] and this upstream patch fixes it, so backport it to kernel 6.12.

It also needs to go to 6.13.y too, right? Please provide backports for both as we can not take them only for older kernels as you would have a regression when upgrading.

thanks,

greg k-h

Zenm Chen

author

2025-02-11 16:34 UTC #2 lore raw

Quoted 3 lines

It also needs to go to 6.13.y too, right? Please provide backports for both as we can not take them only for older kernels as you would have a regression when upgrading.

Hi Greg,

Thanks for reminding! The patch for kernel 6.13 is here:
https://lore.kernel.org/stable/[email protected]/T/#u

Tested with RTL8852BE on Arch Linux (kernel version: 6.13.2-arch1-1) and it did fix the issue.

thanks!

Quoted 4 lines

thanks,
greg k-h

Greg KH

maintainer

2025-02-13 12:32 UTC #3 lore raw

Quoted 11 lines · Zenm Chen

Quoted 3 lines
It also needs to go to 6.13.y too, right? Please provide backports for both as we can not take them only for older kernels as you would have a regression when upgrading.
Hi Greg,
Thanks for reminding! The patch for kernel 6.13 is here:
https://lore.kernel.org/stable/[email protected]/T/#u
Tested with RTL8852BE on Arch Linux (kernel version: 6.13.2-arch1-1) and it did fix the issue.

Ok, thanks, but can you now resend the 6.12.y version?

greg k-h

Zenm Chen

author

2025-02-13 14:36 UTC #4 lore raw

Quoted 6 lines

Quoted 6 lines
Quoted 3 lines
It also needs to go to 6.13.y too, right? Please provide backports for both as we can not take them only for older kernels as you would have a regression when upgrading.
Hi Greg,

Quoted 7 lines

Quoted 5 lines
Thanks for reminding! The patch for kernel 6.13 is here:
https://lore.kernel.org/stable/[email protected]/T/#u
Tested with RTL8852BE on Arch Linux (kernel version: 6.13.2-arch1-1) and it did fix the issue.
Ok, thanks, but can you now resend the 6.12.y version?

I resent the patch for kernel 6.12.y, please tell me if I did it incorrectly, thanks.

https://lore.kernel.org/stable/[email protected]/T/#u