From c7dc847318b1730262fa05193198adbe444b84ad Mon Sep 17 00:00:00 2001 From: Rajeev Kumar Date: Wed, 19 Jun 2019 18:13:51 -0700 Subject: [PATCH] qcacld-3.0: Do not clear wow applicable flag on resume Currently both system and runtime PM resume are clearing wow applicable flag and when runtime PM suspend and suspend are happening in parallel wow applicable flag value is lost and host is by mistake enabling pdev suspend instead of wow suspend. Move the logic to clear wow applicable flag to suspend configuration path such that every time when either runtime pm or system suspend is enabled correct wow parameter is chosen. Change-Id: I821dd5cd23493611dfc5a1c3ebf706defe785b6d CRs-Fixed: 2475227 --- components/pmo/core/src/wlan_pmo_suspend_resume.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/components/pmo/core/src/wlan_pmo_suspend_resume.c b/components/pmo/core/src/wlan_pmo_suspend_resume.c index 4e0492a2e013..ceb1b15c3469 100644 --- a/components/pmo/core/src/wlan_pmo_suspend_resume.c +++ b/components/pmo/core/src/wlan_pmo_suspend_resume.c @@ -446,6 +446,9 @@ static QDF_STATUS pmo_core_psoc_configure_suspend(struct wlan_objmgr_psoc *psoc, pmo_core_configure_dynamic_wake_events(psoc); pmo_core_update_wow_enable(psoc_ctx, true); pmo_core_update_wow_enable_cmd_sent(psoc_ctx, false); + } else { + pmo_debug("Non WOW PDEV Suspend"); + pmo_core_update_wow_enable(psoc_ctx, false); } /* @@ -1159,7 +1162,6 @@ QDF_STATUS pmo_core_psoc_disable_wow_in_fw(struct wlan_objmgr_psoc *psoc, if (ret != QDF_STATUS_SUCCESS) goto out; - pmo_core_update_wow_enable(psoc_ctx, false); pmo_core_update_wow_enable_cmd_sent(psoc_ctx, false); /* To allow the tx pause/unpause events */