commit d400d96b172a5b4a24731a760ac72e9a55162269
author Sravan Kumar Kairam <sgoud@codeaurora.org> Wed Dec 13 12:29:48 2017 +0530
committer Gerrit - the friendly Code Review server <code-review@localhost> Thu Dec 14 21:55:31 2017 -0800
tree 830086da93e5a5df3cae198b98a7c2ed70421dcb
parent f5f0e0843420828be4b06b1d38ee6516aad8a0c9

wlan: Reset the WDA completion event

Currently while sending the WDI HAL DUMP command request WDA
completion event is not reset. This leads to synchronization
mismatch between command send and response received as the
event will be always be set after first command. In this change
reset the completion event before send WDI HAL DUMP command.

Change-Id: Iea98b5e2146abb79d9150e6ebe21d94fc70ed647
CRs-Fixed: 2157735

CORE/WDA/src/wlan_qct_wda.c

diff --git a/CORE/WDA/src/wlan_qct_wda.c b/CORE/WDA/src/wlan_qct_wda.c
index ad17576..56b0879 100644
--- a/CORE/WDA/src/wlan_qct_wda.c
+++ b/CORE/WDA/src/wlan_qct_wda.c
@@ -14014,8 +14014,9 @@
       return ;
    }
 
-   VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_INFO,
-              "%s: WDA HAL DUMP Resp Received",__func__);
+   VOS_TRACE(VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_INFO,
+             "%s: WDA HALDUMP Rsp Received async: %d",
+             __func__, pWdaParams->wdaHALDumpAsync);
 
    pWDA = pWdaParams->pWdaContext;
    buffer = (tANI_U8 *)pWdaParams->wdaMsgParam;
@@ -14073,6 +14074,15 @@
                            "%s: WDA Context Null", __func__);
       return VOS_STATUS_E_RESOURCES;
    }
+
+   vStatus = vos_event_reset(&(pVosContext->wdaCompleteEvent));
+   if (VOS_STATUS_SUCCESS != vStatus)
+   {
+      VOS_TRACE(VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
+                "%s: Event reset failed - status %d", __func__, vStatus);
+      return VOS_STATUS_E_FAILURE;
+   }
+
    pWdaParams = (tWDA_HalDumpReqParams *)vos_mem_malloc(sizeof(tWDA_HalDumpReqParams)) ;
    if(NULL == pWdaParams)
    {

CORE/WDI/CP/src/wlan_qct_wdi.c

diff --git a/CORE/WDI/CP/src/wlan_qct_wdi.c b/CORE/WDI/CP/src/wlan_qct_wdi.c
index 40a9c80..4cadad1 100644
--- a/CORE/WDI/CP/src/wlan_qct_wdi.c
+++ b/CORE/WDI/CP/src/wlan_qct_wdi.c
@@ -22898,6 +22898,9 @@
      return WDI_STATUS_E_FAILURE;
   }
 
+  WPAL_TRACE(eWLAN_MODULE_DAL_CTRL, eWLAN_PAL_TRACE_LEVEL_WARN,
+             "%s: WDI process HAL dump cmd rsp", __func__);
+
   wdiHALDumpCmdRspCb = (WDI_HALDumpCmdRspCb)pWDICtx->pfncRspCB;
 
   /*Initialize the WDI Response structure */