diff --git a/core/sme/src/csr/csr_tdls_process.c b/core/sme/src/csr/csr_tdls_process.c index 3c7791cbcf42..e11fac065138 100644 --- a/core/sme/src/csr/csr_tdls_process.c +++ b/core/sme/src/csr/csr_tdls_process.c @@ -25,14 +25,11 @@ * to the Linux Foundation. */ -/** ------------------------------------------------------------------------- * - ------------------------------------------------------------------------- * - - \file csr_tdls_process.c - - Implementation for the TDLS interface to PE. - ========================================================================== */ - +/* + * DOC: csr_tdls_process.c + * + * Implementation for the TDLS interface to PE. + */ #ifdef FEATURE_WLAN_TDLS #include "ani_global.h" /* for tpAniSirGlobal */ @@ -256,7 +253,7 @@ QDF_STATUS csr_tdls_send_link_establish_params(tHalHandle hHal, tdlsLinkEstablishParams-> supportedOperClassesLen); tdlsLinkEstablishCmdInfo->supportedOperClassesLen = - tdlsLinkEstablishParams->supportedOperClassesLen; + tdlsLinkEstablishParams->supportedOperClassesLen; tdlsLinkEstablishCmdInfo->isResponder = tdlsLinkEstablishParams->isResponder; tdlsLinkEstablishCmdInfo->maxSp = @@ -357,9 +354,9 @@ static QDF_STATUS tdls_send_message(tpAniSirGlobal pMac, uint16_t msg_type, { tSirMbMsg *pMsg = (tSirMbMsg *) msg_data; + pMsg->type = msg_type; pMsg->msgLen = (uint16_t) (msg_size); - QDF_TRACE(QDF_MODULE_ID_SME, QDF_TRACE_LEVEL_DEBUG, ("sending msg = %d"), pMsg->type); if (umac_send_mb_message_to_mac(pMsg) != QDF_STATUS_SUCCESS) { @@ -400,7 +397,9 @@ static QDF_STATUS csr_tdls_process_send_mgmt(tpAniSirGlobal pMac, tSmeCmd *cmd) return status; } tdlsSendMgmtReq->sessionId = cmd->sessionId; - /* Using dialog as transactionId. This can be used to match response with request */ + /* Using dialog as transactionId. This can be used to match + * response with request + */ tdlsSendMgmtReq->transactionId = tdlsSendMgmtCmdInfo->dialog; tdlsSendMgmtReq->reqType = tdlsSendMgmtCmdInfo->frameType; tdlsSendMgmtReq->dialog = tdlsSendMgmtCmdInfo->dialog; @@ -425,9 +424,9 @@ static QDF_STATUS csr_tdls_process_send_mgmt(tpAniSirGlobal pMac, tSmeCmd *cmd) (void *)tdlsSendMgmtReq, sizeof(tSirTdlsSendMgmtReq) + tdlsSendMgmtCmdInfo->len); - if (!QDF_IS_STATUS_SUCCESS(status)) { + if (!QDF_IS_STATUS_SUCCESS(status)) sme_err("Failed to send request to MAC"); - } + if (tdlsSendMgmtCmdInfo->len && tdlsSendMgmtCmdInfo->buf) { /* Done with the buf. Free it. */ qdf_mem_free(tdlsSendMgmtCmdInfo->buf); @@ -468,7 +467,9 @@ static QDF_STATUS csr_tdls_process_add_sta(tpAniSirGlobal pMac, tSmeCmd *cmd) } tdlsAddStaReq->sessionId = cmd->sessionId; tdlsAddStaReq->tdlsAddOper = tdlsAddStaCmdInfo->tdlsAddOper; - /* Using dialog as transactionId. This can be used to match response with request */ + /* Using dialog as transactionId. This can be used to match + * response with request + */ tdlsAddStaReq->transactionId = 0; qdf_mem_copy(tdlsAddStaReq->bssid.bytes, @@ -501,9 +502,9 @@ static QDF_STATUS csr_tdls_process_add_sta(tpAniSirGlobal pMac, tSmeCmd *cmd) status = tdls_send_message(pMac, eWNI_SME_TDLS_ADD_STA_REQ, (void *)tdlsAddStaReq, sizeof(tSirTdlsAddStaReq)); - if (!QDF_IS_STATUS_SUCCESS(status)) { + if (!QDF_IS_STATUS_SUCCESS(status)) sme_err("Failed to send request to MAC"); - } + return status; } @@ -536,7 +537,9 @@ static QDF_STATUS csr_tdls_process_del_sta(tpAniSirGlobal pMac, tSmeCmd *cmd) return status; } tdlsDelStaReq->sessionId = cmd->sessionId; - /* Using dialog as transactionId. This can be used to match response with request */ + /* Using dialog as transactionId. This can be used to match + * response with request + */ tdlsDelStaReq->transactionId = 0; qdf_mem_copy(tdlsDelStaReq->bssid.bytes, @@ -550,9 +553,9 @@ static QDF_STATUS csr_tdls_process_del_sta(tpAniSirGlobal pMac, tSmeCmd *cmd) status = tdls_send_message(pMac, eWNI_SME_TDLS_DEL_STA_REQ, (void *)tdlsDelStaReq, sizeof(tSirTdlsDelStaReq)); - if (!QDF_IS_STATUS_SUCCESS(status)) { + if (!QDF_IS_STATUS_SUCCESS(status)) sme_err("Failed to send request to MAC"); - } + return status; } @@ -563,46 +566,40 @@ QDF_STATUS csr_tdls_process_cmd(tpAniSirGlobal pMac, tSmeCmd *cmd) { eSmeCommandType cmdType = cmd->command; bool status = true; + switch (cmdType) { case eSmeCommandTdlsSendMgmt: { status = csr_tdls_process_send_mgmt(pMac, cmd); - if (QDF_IS_STATUS_SUCCESS(status)) { + if (QDF_IS_STATUS_SUCCESS(status)) status = false; - } } break; case eSmeCommandTdlsAddPeer: { status = csr_tdls_process_add_sta(pMac, cmd); - if (QDF_IS_STATUS_SUCCESS(status)) { + if (QDF_IS_STATUS_SUCCESS(status)) status = false; - } } break; case eSmeCommandTdlsDelPeer: { status = csr_tdls_process_del_sta(pMac, cmd); - if (QDF_IS_STATUS_SUCCESS(status)) { + if (QDF_IS_STATUS_SUCCESS(status)) status = false; - } } break; case eSmeCommandTdlsLinkEstablish: { status = csr_tdls_process_link_establish(pMac, cmd); - if (QDF_IS_STATUS_SUCCESS(status)) { + if (QDF_IS_STATUS_SUCCESS(status)) status = false; - } } break; default: - { /* TODO: Add defualt handling */ break; } - - } return status; } @@ -626,7 +623,9 @@ QDF_STATUS csr_tdls_process_link_establish(tpAniSirGlobal pMac, tSmeCmd *cmd) return QDF_STATUS_E_NOMEM; } tdlsLinkEstablishReq->sessionId = cmd->sessionId; - /* Using dialog as transactionId. This can be used to match response with request */ + /* Using dialog as transactionId. This can be used to match + * response with request + */ tdlsLinkEstablishReq->transactionId = 0; qdf_copy_macaddr(&tdlsLinkEstablishReq->peermac, &tdlsLinkEstablishCmdInfo->peermac); @@ -828,7 +827,7 @@ QDF_STATUS tdls_msg_processor(tpAniSirGlobal pMac, uint16_t msgType, tevent->peer_reason); csr_roam_call_callback(pMac, tevent->sessionId, &roamInfo, 0, eCSR_ROAM_TDLS_STATUS_UPDATE, - eCSR_ROAM_RESULT_TDLS_CONNECTION_TRACKER_NOTIFICATION); + eCSR_ROAM_RESULT_TDLS_CONNECTION_TRACKER_NOTIFICATION); break; default: break;