wlan: WDA: removing newlines from the debug logs
dmesg log is a limited and extremely precious
debugging tool. Any unnecessary prints are avoided.
Change-Id: I6bae68d227b60f73f3f0536befa595dde1e51a0d
CRs-Fixed: 540451
diff --git a/CORE/WDA/src/wlan_qct_wda.c b/CORE/WDA/src/wlan_qct_wda.c
index 6d6e3bd..af81709 100644
--- a/CORE/WDA/src/wlan_qct_wda.c
+++ b/CORE/WDA/src/wlan_qct_wda.c
@@ -260,7 +260,7 @@
if(!VOS_IS_STATUS_SUCCESS(status))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "WDI Sync Event init failed - status = %d\n", status);
+ "WDI Sync Event init failed - status = %d", status);
goto error;
}
/* Init Frame transfer event */
@@ -268,21 +268,21 @@
if(!VOS_IS_STATUS_SUCCESS(status))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "VOS Mgmt Frame Event init failed - status = %d\n", status);
+ "VOS Mgmt Frame Event init failed - status = %d", status);
goto error;
}
status = vos_event_init(&wdaContext->suspendDataTxEvent);
if(!VOS_IS_STATUS_SUCCESS(status))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "VOS suspend data tx Event init failed - status = %d\n", status);
+ "VOS suspend data tx Event init failed - status = %d", status);
goto error;
}
status = vos_event_init(&wdaContext->waitOnWdiIndicationCallBack);
if(!VOS_IS_STATUS_SUCCESS(status))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "VOS wait On Wdi Ind Event init failed - status = %d\n", status);
+ "VOS wait On Wdi Ind Event init failed - status = %d", status);
goto error;
}
vos_trace_setLevel(VOS_MODULE_ID_WDA,VOS_TRACE_LEVEL_ERROR);
@@ -1889,7 +1889,7 @@
if(!VOS_IS_STATUS_SUCCESS(vstatus))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "WDI Sync Event destroy failed - status = %d\n", status);
+ "WDI Sync Event destroy failed - status = %d", status);
status = VOS_STATUS_E_FAILURE;
}
@@ -1897,21 +1897,21 @@
if(!VOS_IS_STATUS_SUCCESS(vstatus))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "VOS Event destroy failed - status = %d\n", status);
+ "VOS Event destroy failed - status = %d", status);
status = VOS_STATUS_E_FAILURE;
}
vstatus = vos_event_destroy(&wdaContext->suspendDataTxEvent);
if(!VOS_IS_STATUS_SUCCESS(vstatus))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "VOS Event destroy failed - status = %d\n", status);
+ "VOS Event destroy failed - status = %d", status);
status = VOS_STATUS_E_FAILURE;
}
vstatus = vos_event_destroy(&wdaContext->waitOnWdiIndicationCallBack);
if(!VOS_IS_STATUS_SUCCESS(vstatus))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "VOS Event destroy failed - status = %d\n", status);
+ "VOS Event destroy failed - status = %d", status);
status = VOS_STATUS_E_FAILURE;
}
/* free WDA context */
@@ -2191,14 +2191,14 @@
if(!VOS_IS_STATUS_SUCCESS(vosStatus))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "NEW VOS Event Set failed - status = %d \n", vosStatus);
+ "NEW VOS Event Set failed - status = %d", vosStatus);
}
/* If TL suspended had timedout before this callback was called, resume back
* TL.*/
if (pWDA->txSuspendTimedOut)
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "Late TLSuspendCallback, resuming TL back again\n");
+ "Late TLSuspendCallback, resuming TL back again");
WDA_ResumeDataTx(pWDA);
pWDA->txSuspendTimedOut = FALSE;
}
@@ -2219,7 +2219,7 @@
if (pWDA->txSuspendTimedOut)
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "TL suspend timedout previously, CB not called yet\n");
+ "TL suspend timedout previously, CB not called yet");
return status;
}
/* Reset the event to be not signalled */
@@ -2227,7 +2227,7 @@
if(!VOS_IS_STATUS_SUCCESS(status))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "VOS Event reset failed - status = %d\n",status);
+ "VOS Event reset failed - status = %d",status);
return VOS_STATUS_E_FAILURE;
}
/*Indicate TL to suspend transmission for all Sta Id */
@@ -4146,7 +4146,7 @@
default:
{
VOS_TRACE(VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "There is no HAL CFG Id corresponding to WNI CFG Id: %d\n",
+ "There is no HAL CFG Id corresponding to WNI CFG Id: %d",
wniCfgId);
return VOS_STATUS_E_INVAL;
}
@@ -4170,7 +4170,7 @@
if(WDI_STATUS_SUCCESS != wdiStatus)
{
VOS_TRACE(VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "%s: CFG (%d) config failure \n", __func__,
+ "%s: CFG (%d) config failure", __func__,
((tHalCfg *)(wdiCfgParam->pConfigBuffer))->type);
}
@@ -4227,7 +4227,7 @@
if(NULL == wdiCfgReqParam->pConfigBuffer)
{
VOS_TRACE(VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "%s: VOS MEM Alloc Failure \n", __func__);
+ "%s: VOS MEM Alloc Failure", __func__);
vos_mem_free(wdiCfgReqParam);
VOS_ASSERT(0);
return VOS_STATUS_E_NOMEM;
@@ -4242,7 +4242,7 @@
&val) != eSIR_SUCCESS)
{
VOS_TRACE(VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "Failed to cfg get id %d\n", cfgParam->bodyval);
+ "Failed to cfg get id %d", cfgParam->bodyval);
vos_mem_free(wdiCfgReqParam->pConfigBuffer);
vos_mem_free(wdiCfgReqParam);
return eSIR_FAILURE;
@@ -4309,7 +4309,7 @@
&val ))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "WEP Key index [%d] may not configured in CFG\n",i);
+ "WEP Key index [%d] may not configured in CFG",i);
}
else
{
@@ -5600,7 +5600,7 @@
((WLANTL_STA_CONNECTED != tlSTAState) && (WLANTL_STA_AUTHENTICATED != tlSTAState)))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "Peer staIdx %d hasn't established yet(%d). Send ADD BA failure to PE.\n", pAddBAReqParams->staIdx, tlSTAState );
+ "Peer staIdx %d hasn't established yet(%d). Send ADD BA failure to PE.", pAddBAReqParams->staIdx, tlSTAState );
status = WDI_STATUS_E_NOT_ALLOWED;
pAddBAReqParams->status =
CONVERT_WDI2SIR_STATUS(status) ;
@@ -5618,9 +5618,9 @@
if(IS_WDI_STATUS_FAILURE(status))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "Failure in ADD BA Session REQ Params WDI API, free all the memory =%d\n", status);
+ "Failure in ADD BA Session REQ Params WDI API, free all the memory =%d", status);
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "Send ADD BA failure response to PE\n");
+ "Send ADD BA failure response to PE");
pAddBAReqParams->status =
CONVERT_WDI2SIR_STATUS(status) ;
WDA_SendMsg(pWDA, WDA_ADDBA_RSP, (void *)pAddBAReqParams , 0) ;
@@ -7580,7 +7580,7 @@
if(NULL == configParam)
{
VOS_TRACE(VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "%s: VOS MEM Alloc Failure \n", __func__);
+ "%s: VOS MEM Alloc Failure", __func__);
VOS_ASSERT(0);
vos_mem_free(pWdaParams);
vos_mem_free(wdiPowerSaveCfg);
@@ -7943,7 +7943,7 @@
if(WDI_STATUS_SUCCESS != wdiStatus)
{
VOS_TRACE(VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "%s: RXP config filter failure \n", __func__ );
+ "%s: RXP config filter failure", __func__ );
}
if(NULL == pWdaParams)
{
@@ -8142,7 +8142,7 @@
if(WDI_STATUS_SUCCESS != resumeRspParams->wdiStatus)
{
VOS_TRACE(VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "%s: Process Wlan Resume failure \n", __func__ );
+ "%s: Process Wlan Resume failure", __func__ );
}
vos_mem_free(pWdaParams->wdaWdiApiMsgParam) ;
vos_mem_free(pWdaParams->wdaMsgParam);
@@ -10413,12 +10413,12 @@
if ( vStatus == VOS_STATUS_E_TIMEOUT )
{
VOS_TRACE( VOS_MODULE_ID_VOSS, VOS_TRACE_LEVEL_ERROR,
- "%s: Timeout occurred before WDA_HALDUMP complete\n",__func__);
+ "%s: Timeout occurred before WDA_HALDUMP complete",__func__);
}
else
{
VOS_TRACE( VOS_MODULE_ID_VOSS, VOS_TRACE_LEVEL_ERROR,
- "%s: WDA_HALDUMP reporting other error \n",__func__);
+ "%s: WDA_HALDUMP reporting other error",__func__);
}
VOS_ASSERT(0);
}
@@ -10925,7 +10925,7 @@
if(!VOS_IS_STATUS_SUCCESS(status))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "NEW VOS Event Set failed - status = %d \n", status);
+ "NEW VOS Event Set failed - status = %d", status);
}
return status;
}
@@ -10960,7 +10960,7 @@
}
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_INFO_HIGH,
- "Tx Mgmt Frame Subtype: %d alloc(%p)\n", pFc->subType, pFrmBuf);
+ "Tx Mgmt Frame Subtype: %d alloc(%p)", pFc->subType, pFrmBuf);
pMac = (tpAniSirGlobal )VOS_GET_MAC_CTXT(pWDA->pVosContext);
if(NULL == pMac)
{
@@ -10981,7 +10981,7 @@
{
/* Already TxComp is active no need to active again */
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "There is already one request pending for tx complete\n");
+ "There is already one request pending for tx complete");
pWDA->pAckTxCbFunc( pMac, 0);
pWDA->pAckTxCbFunc = NULL;
@@ -11014,7 +11014,7 @@
if(!VOS_IS_STATUS_SUCCESS(status))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "VOS Event reset failed - status = %d\n",status);
+ "VOS Event reset failed - status = %d",status);
pCompFunc(pWDA->pVosContext, (vos_pkt_t *)pFrmBuf);
if( pAckTxComp )
{
@@ -11084,7 +11084,7 @@
WDA_TxComplete, NULL, txFlag)) != VOS_STATUS_SUCCESS)
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "Sending Mgmt Frame failed - status = %d\n", status);
+ "Sending Mgmt Frame failed - status = %d", status);
pCompFunc(VOS_GET_MAC_CTXT(pWDA->pVosContext), (vos_pkt_t *)pFrmBuf);
vos_atomic_set_U32(&pWDA->VosPacketToFree, (v_U32_t)WDA_TX_PACKET_FREED);/*reset the VosPacket_freed*/
if( pAckTxComp )
@@ -11828,11 +11828,11 @@
WDA_ProcessUpdateOpMode(pWDA, (tUpdateVHTOpMode *)pMsg->bodyptr);
else
VOS_TRACE(VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- " VHT OpMode Feature is Not Supported \n");
+ " VHT OpMode Feature is Not Supported");
}
else
VOS_TRACE(VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- " 11AC Feature is Not Supported \n");
+ " 11AC Feature is Not Supported");
break;
}
#endif
@@ -13015,14 +13015,14 @@
if( pWDA->pAckTxCbFunc )
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "TxComplete timer expired\n");
+ "TxComplete timer expired");
pWDA->pAckTxCbFunc( pMac, 0);
pWDA->pAckTxCbFunc = NULL;
}
else
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "There is no request pending for TxComplete and wait timer expired\n");
+ "There is no request pending for TxComplete and wait timer expired");
}
}
/*
@@ -14026,19 +14026,19 @@
&pRcvPktFilterCfg->paramsData[i],
sizeof(pwdiSetRcvPktFilterReqParamsType->wdiPktFilterCfg.paramsData[i]));
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_INFO,
- "Proto %d Comp Flag %d \n",
+ "Proto %d Comp Flag %d",
pwdiSetRcvPktFilterReqParamsType->
wdiPktFilterCfg.paramsData[i].protocolLayer,
pwdiSetRcvPktFilterReqParamsType->
wdiPktFilterCfg.paramsData[i].cmpFlag);
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_INFO,
- "Data Offset %d Data Len %d\n",
+ "Data Offset %d Data Len %d",
pwdiSetRcvPktFilterReqParamsType->
wdiPktFilterCfg.paramsData[i].dataOffset,
pwdiSetRcvPktFilterReqParamsType->
wdiPktFilterCfg.paramsData[i].dataLength);
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_INFO,
- "CData: %d:%d:%d:%d:%d:%d\n",
+ "CData: %d:%d:%d:%d:%d:%d",
pwdiSetRcvPktFilterReqParamsType->
wdiPktFilterCfg.paramsData[i].compareData[0],
pwdiSetRcvPktFilterReqParamsType->
@@ -14052,7 +14052,7 @@
pwdiSetRcvPktFilterReqParamsType->
wdiPktFilterCfg.paramsData[i].compareData[5]);
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_INFO,
- "MData: %d:%d:%d:%d:%d:%d\n",
+ "MData: %d:%d:%d:%d:%d:%d",
pwdiSetRcvPktFilterReqParamsType->
wdiPktFilterCfg.paramsData[i].dataMask[0],
pwdiSetRcvPktFilterReqParamsType->
@@ -14640,21 +14640,21 @@
if(!VOS_IS_STATUS_SUCCESS(status))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "VOS Event destroy failed - status = %d\n", status);
+ "VOS Event destroy failed - status = %d", status);
status = VOS_STATUS_E_FAILURE;
}
status = vos_event_destroy(&pWDA->suspendDataTxEvent);
if(!VOS_IS_STATUS_SUCCESS(status))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "VOS Event destroy failed - status = %d\n", status);
+ "VOS Event destroy failed - status = %d", status);
status = VOS_STATUS_E_FAILURE;
}
status = vos_event_destroy(&pWDA->waitOnWdiIndicationCallBack);
if(!VOS_IS_STATUS_SUCCESS(status))
{
VOS_TRACE( VOS_MODULE_ID_WDA, VOS_TRACE_LEVEL_ERROR,
- "VOS Event destroy failed - status = %d\n", status);
+ "VOS Event destroy failed - status = %d", status);
status = VOS_STATUS_E_FAILURE;
}
/* free WDA context */