wlan: HDD: removing newlines from the debug logs
dmesg log is a limited and extremely precious
debugging tool. Any unnecessary prints are avoided.
Change-Id: I94c77696af432b856967a7a0d628c06f2b3f8679
CRs-Fixed: 540451
diff --git a/CORE/HDD/src/wlan_hdd_main.c b/CORE/HDD/src/wlan_hdd_main.c
index 07bfaa3..0721b28 100644
--- a/CORE/HDD/src/wlan_hdd_main.c
+++ b/CORE/HDD/src/wlan_hdd_main.c
@@ -1693,7 +1693,7 @@
sizeof(tSirMacAddr)))
{
VOS_TRACE( VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_FATAL,
- "%s: failed to copy data to user buffer\n", __func__);
+ "%s: failed to copy data to user buffer", __func__);
ret = -EFAULT;
}
}
@@ -2792,7 +2792,7 @@
wlan_hdd_validate_operation_channel(pAdapter, channel))
{
hddLog(VOS_TRACE_LEVEL_ERROR,
- "%s: Invalid Channel [%d] \n", __func__, channel);
+ "%s: Invalid Channel [%d]", __func__, channel);
return -EINVAL;
}
@@ -4906,7 +4906,7 @@
struct completion *completion_var = (struct completion*) callbackContext;
- hddLog(VOS_TRACE_LEVEL_ERROR, "HDD BMPS request Callback, status = %d\n", status);
+ hddLog(VOS_TRACE_LEVEL_ERROR, "HDD BMPS request Callback, status = %d", status);
if(completion_var != NULL)
{
complete(completion_var);
@@ -5412,7 +5412,7 @@
if(!rc)
{
VOS_TRACE(VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_ERROR,
- ("ERROR: HDD Wait for Action Confirmation Failed!!\n"));
+ ("ERROR: HDD Wait for Action Confirmation Failed!!"));
}
}
return;
@@ -5694,7 +5694,7 @@
if(eHAL_STATUS_SUCCESS != halStatus)
{
status = VOS_STATUS_E_FAILURE;
- hddLog(VOS_TRACE_LEVEL_ERROR,"%s: Fail to Disable Power Save\n", __func__);
+ hddLog(VOS_TRACE_LEVEL_ERROR,"%s: Fail to Disable Power Save", __func__);
VOS_ASSERT(0);
return status;
}
@@ -5707,7 +5707,7 @@
if(eHAL_STATUS_SUCCESS != halStatus)
{
status = VOS_STATUS_E_FAILURE;
- hddLog(VOS_TRACE_LEVEL_ERROR,"%s: Fail to Stop Auto Bmps Timer\n", __func__);
+ hddLog(VOS_TRACE_LEVEL_ERROR,"%s: Fail to Stop Auto Bmps Timer", __func__);
VOS_ASSERT(0);
return status;
}
@@ -5733,7 +5733,7 @@
else
{
status = VOS_STATUS_E_FAILURE;
- hddLog(VOS_TRACE_LEVEL_ERROR,"%s: Request for Full Power failed\n", __func__);
+ hddLog(VOS_TRACE_LEVEL_ERROR,"%s: Request for Full Power failed", __func__);
VOS_ASSERT(0);
return status;
}
@@ -5756,7 +5756,7 @@
VOS_STATUS status = VOS_STATUS_E_FAILURE;
VOS_STATUS exitbmpsStatus;
- hddLog(VOS_TRACE_LEVEL_INFO_HIGH, "%s iface =%s type = %d\n",__func__,iface_name,session_type);
+ hddLog(VOS_TRACE_LEVEL_INFO_HIGH, "%s iface =%s type = %d",__func__,iface_name,session_type);
//Disable BMPS incase of Concurrency
exitbmpsStatus = hdd_disable_bmps_imps(pHddCtx, session_type);
@@ -6070,7 +6070,7 @@
eANI_BOOLEAN_FALSE) )
{
hddLog(LOGE,
- "Could not pass on WNI_CFG_PROBE_RSP_ADDNIE_DATA1 to CCM\n");
+ "Could not pass on WNI_CFG_PROBE_RSP_ADDNIE_DATA1 to CCM");
}
if ( eHAL_STATUS_FAILURE == ccmCfgSetStr((WLAN_HDD_GET_CTX(pHostapdAdapter))->hHal,
@@ -6078,7 +6078,7 @@
eANI_BOOLEAN_FALSE) )
{
hddLog(LOGE,
- "Could not pass on WNI_CFG_PROBE_RSP_ADDNIE_DATA2 to CCM\n");
+ "Could not pass on WNI_CFG_PROBE_RSP_ADDNIE_DATA2 to CCM");
}
if ( eHAL_STATUS_FAILURE == ccmCfgSetStr((WLAN_HDD_GET_CTX(pHostapdAdapter))->hHal,
@@ -6086,7 +6086,7 @@
eANI_BOOLEAN_FALSE) )
{
hddLog(LOGE,
- "Could not pass on WNI_CFG_PROBE_RSP_ADDNIE_DATA3 to CCM\n");
+ "Could not pass on WNI_CFG_PROBE_RSP_ADDNIE_DATA3 to CCM");
}
}
@@ -6799,7 +6799,7 @@
break;
memset(&(pAdapter->mc_addr_list.addr[i][0]), 0, ETH_ALEN);
memcpy(&(pAdapter->mc_addr_list.addr[i][0]), ha->addr, ETH_ALEN);
- hddLog(VOS_TRACE_LEVEL_INFO, "\n%s: mlist[%d] = "MAC_ADDRESS_STR,
+ hddLog(VOS_TRACE_LEVEL_INFO, "%s: mlist[%d] = "MAC_ADDRESS_STR,
__func__, i,
MAC_ADDR_ARRAY(pAdapter->mc_addr_list.addr[i]));
i++;
@@ -7189,12 +7189,12 @@
* exited at this point
*/
hddLog(VOS_TRACE_LEVEL_WARN, "In module exit: Cancel the vote for XO Core ON"
- " when WLAN is turned OFF\n");
+ " when WLAN is turned OFF");
if (vos_chipVoteXOCore(NULL, NULL, NULL, VOS_FALSE) != VOS_STATUS_SUCCESS)
{
hddLog(VOS_TRACE_LEVEL_ERROR, "Could not cancel the vote for XO Core ON."
" Not returning failure."
- " Power consumed will be high\n");
+ " Power consumed will be high");
}
hdd_close_all_adapters( pHddCtx );
@@ -7250,20 +7250,20 @@
status = vos_nv_getValidity(VNV_FIELD_IMAGE, &itemIsValid);
if(status != VOS_STATUS_SUCCESS)
{
- hddLog(VOS_TRACE_LEVEL_ERROR," vos_nv_getValidity() failed\n ");
+ hddLog(VOS_TRACE_LEVEL_ERROR," vos_nv_getValidity() failed");
return VOS_STATUS_E_FAILURE;
}
if (itemIsValid == VOS_TRUE)
{
- hddLog(VOS_TRACE_LEVEL_INFO_HIGH," Reading the Macaddress from NV\n ");
+ hddLog(VOS_TRACE_LEVEL_INFO_HIGH," Reading the Macaddress from NV");
status = vos_nv_readMultiMacAddress((v_U8_t *)&macFromNV[0].bytes[0],
VOS_MAX_CONCURRENCY_PERSONA);
if(status != VOS_STATUS_SUCCESS)
{
/* Get MAC from NV fail, not update CFG info
* INI MAC value will be used for MAC setting */
- hddLog(VOS_TRACE_LEVEL_ERROR," vos_nv_readMacAddress() failed\n ");
+ hddLog(VOS_TRACE_LEVEL_ERROR," vos_nv_readMacAddress() failed");
return VOS_STATUS_E_FAILURE;
}
@@ -8421,7 +8421,7 @@
if (vos_chipVoteXOCore(NULL, NULL, NULL, VOS_FALSE) != VOS_STATUS_SUCCESS)
{
hddLog(VOS_TRACE_LEVEL_ERROR, "Could not cancel XO Core ON vote. Not returning failure."
- " Power consumed will be high\n");
+ " Power consumed will be high");
}
} while (0);