[PATCH] ath10k: fix S5 power consumption issue for QCA9377

[PATCH] ath10k: fix S5 power consumption issue for QCA9377

[Yu Wang]

After system entering S5 (shut down but system still
providing power to QCA9377) on Ubuntu platform, power
consumption of QCA9377 is 69mA, which is too high.
The root cause is pci_soft_reset is not set for QCA9377
during pci probe.
To fix this issue, set 'pci_soft_reset' to 'th10k_pci_warm_reset',
and then the power consumption drops to a normal value(10mA).
Verified on Dell Ubuntu platform with firmware:
WLAN.TF.1.0-00002-QCATFSWPZ-5

Signed-off-by: Yu Wang <yyuwang@codeaurora.org>
---
 drivers/net/wireless/ath/ath10k/pci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/ath/ath10k/pci.c b/drivers/net/wireless/ath/ath10k/pci.c
index 01b4edb..0a4ce98 100644
--- a/drivers/net/wireless/ath/ath10k/pci.c
+++ b/drivers/net/wireless/ath/ath10k/pci.c
@@ -3554,7 +3554,7 @@ static int ath10k_pci_probe(struct pci_dev *pdev,
 	case QCA9377_1_0_DEVICE_ID:
 		hw_rev = ATH10K_HW_QCA9377;
 		pci_ps = true;
-		pci_soft_reset = NULL;
+		pci_soft_reset = ath10k_pci_warm_reset;
 		pci_hard_reset = ath10k_pci_qca6174_chip_reset;
 		targ_cpu_to_ce_addr = ath10k_pci_qca6174_targ_cpu_to_ce_addr;
 		break;
-- 
1.9.1

RE: [PATCH] ath10k: fix S5 power consumption issue for QCA9377

[Robert Chang]

+ Karen

Thanks,
Robert

-----Original Message-----
From: ath10k <ath10k-bounces@lists.infradead.org> On Behalf Of Yu Wang
Sent: Tuesday, January 15, 2019 2:31 PM
To: ath10k@lists.infradead.org
Cc: Yu Wang <yyuwang@codeaurora.org>; linux-wireless@vger.kernel.org
Subject: [EXT] [PATCH] ath10k: fix S5 power consumption issue for QCA9377

After system entering S5 (shut down but system still providing power to QCA9377) on Ubuntu platform, power consumption of QCA9377 is 69mA, which is too high.
The root cause is pci_soft_reset is not set for QCA9377 during pci probe.
To fix this issue, set 'pci_soft_reset' to 'th10k_pci_warm_reset', and then the power consumption drops to a normal value(10mA).
Verified on Dell Ubuntu platform with firmware:
WLAN.TF.1.0-00002-QCATFSWPZ-5

Signed-off-by: Yu Wang <yyuwang@codeaurora.org>
---
 drivers/net/wireless/ath/ath10k/pci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/ath/ath10k/pci.c b/drivers/net/wireless/ath/ath10k/pci.c
index 01b4edb..0a4ce98 100644
--- a/drivers/net/wireless/ath/ath10k/pci.c
+++ b/drivers/net/wireless/ath/ath10k/pci.c
@@ -3554,7 +3554,7 @@ static int ath10k_pci_probe(struct pci_dev *pdev,
 	case QCA9377_1_0_DEVICE_ID:
 		hw_rev = ATH10K_HW_QCA9377;
 		pci_ps = true;
-		pci_soft_reset = NULL;
+		pci_soft_reset = ath10k_pci_warm_reset;
 		pci_hard_reset = ath10k_pci_qca6174_chip_reset;
 		targ_cpu_to_ce_addr = ath10k_pci_qca6174_targ_cpu_to_ce_addr;
 		break;
--
1.9.1


_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

Re: [PATCH] ath10k: fix S5 power consumption issue for QCA9377

[Kalle Valo]

Yu Wang <yyuwang@codeaurora.org> wrote:

> After system entering S5 (shut down but system still
> providing power to QCA9377) on Ubuntu platform, power
> consumption of QCA9377 is 69mA, which is too high.
> The root cause is pci_soft_reset is not set for QCA9377
> during pci probe.
> To fix this issue, set 'pci_soft_reset' to 'th10k_pci_warm_reset',
> and then the power consumption drops to a normal value(10mA).
> Verified on Dell Ubuntu platform with firmware:
> WLAN.TF.1.0-00002-QCATFSWPZ-5
> 
> Signed-off-by: Yu Wang <yyuwang@codeaurora.org>
> Signed-off-by: Yu Wang <yyuwang@codeaurora.org>
> Signed-off-by: Kalle Valo <kvalo@codeaurora.org>

Patch applied to ath-next branch of ath.git, thanks.

1a86be476cfa ath10k: fix S5 power consumption issue for QCA9377

-- 
https://patchwork.kernel.org/patch/10764017/

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