qcacmn: Fix HTT HAL Tx status mapping
Currently HTT_TX_FW2WBM_TX_STATUS_DROP and HTT_TX_FW2WBM_TX_STATUS_TTL are mapped to HAL_TX_TQM_RR_REM_CMD_REM which results in the wrong Tx status print in case HTT_TX_FW2WBM_TX_STATUS_TTL where No ack is received. So to fix the status log issue map HTT_TX_FW2WBM_TX_STATUS_DROP to HAL_TX_TQM_RR_REM_CMD_REM and HTT_TX_FW2WBM_TX_STATUS_TTL to HAL_TX_TQM_RR_REM_CMD_TX to log correct Tx Fail Status. Change-Id: If125313f6b831ce5ca0f7eec9e8e3f1a03229f16 CRs-Fixed: 3362679
This commit is contained in:

committed by
Madan Koyyalamudi

parent
001bc02d42
commit
877eaf11ea
@@ -331,9 +331,17 @@ void dp_tx_process_htt_completion_be(struct dp_soc *soc,
|
|||||||
ts.tsf = htt_desc[4];
|
ts.tsf = htt_desc[4];
|
||||||
ts.first_msdu = 1;
|
ts.first_msdu = 1;
|
||||||
ts.last_msdu = 1;
|
ts.last_msdu = 1;
|
||||||
ts.status = (tx_status == HTT_TX_FW2WBM_TX_STATUS_OK ?
|
switch (tx_status) {
|
||||||
HAL_TX_TQM_RR_FRAME_ACKED :
|
case HTT_TX_FW2WBM_TX_STATUS_OK:
|
||||||
HAL_TX_TQM_RR_REM_CMD_REM);
|
ts.status = HAL_TX_TQM_RR_FRAME_ACKED;
|
||||||
|
break;
|
||||||
|
case HTT_TX_FW2WBM_TX_STATUS_DROP:
|
||||||
|
ts.status = HAL_TX_TQM_RR_REM_CMD_REM;
|
||||||
|
break;
|
||||||
|
case HTT_TX_FW2WBM_TX_STATUS_TTL:
|
||||||
|
ts.status = HAL_TX_TQM_RR_REM_CMD_TX;
|
||||||
|
break;
|
||||||
|
}
|
||||||
tid = ts.tid;
|
tid = ts.tid;
|
||||||
if (qdf_unlikely(tid >= CDP_MAX_DATA_TIDS))
|
if (qdf_unlikely(tid >= CDP_MAX_DATA_TIDS))
|
||||||
tid = CDP_MAX_DATA_TIDS - 1;
|
tid = CDP_MAX_DATA_TIDS - 1;
|
||||||
|
@@ -178,9 +178,17 @@ void dp_tx_process_htt_completion_li(struct dp_soc *soc,
|
|||||||
ts.tsf = htt_desc[3];
|
ts.tsf = htt_desc[3];
|
||||||
ts.first_msdu = 1;
|
ts.first_msdu = 1;
|
||||||
ts.last_msdu = 1;
|
ts.last_msdu = 1;
|
||||||
ts.status = (tx_status == HTT_TX_FW2WBM_TX_STATUS_OK ?
|
switch (tx_status) {
|
||||||
HAL_TX_TQM_RR_FRAME_ACKED :
|
case HTT_TX_FW2WBM_TX_STATUS_OK:
|
||||||
HAL_TX_TQM_RR_REM_CMD_REM);
|
ts.status = HAL_TX_TQM_RR_FRAME_ACKED;
|
||||||
|
break;
|
||||||
|
case HTT_TX_FW2WBM_TX_STATUS_DROP:
|
||||||
|
ts.status = HAL_TX_TQM_RR_REM_CMD_REM;
|
||||||
|
break;
|
||||||
|
case HTT_TX_FW2WBM_TX_STATUS_TTL:
|
||||||
|
ts.status = HAL_TX_TQM_RR_REM_CMD_TX;
|
||||||
|
break;
|
||||||
|
}
|
||||||
tid = ts.tid;
|
tid = ts.tid;
|
||||||
if (qdf_unlikely(tid >= CDP_MAX_DATA_TIDS))
|
if (qdf_unlikely(tid >= CDP_MAX_DATA_TIDS))
|
||||||
tid = CDP_MAX_DATA_TIDS - 1;
|
tid = CDP_MAX_DATA_TIDS - 1;
|
||||||
|
Reference in New Issue
Block a user