Browse Source

qcacmn: Fix access to rx mpdu start tlv

MPDU start TLV structure is different across pine and hk/cyp. The
access to individual members of this structure is already taken care
in HAL APIs.

Tha addition of offset is redundant and wrongly defined offset values
led to wrong interpretation of mpdu start tlv fields like RSSI in
QCN9000.

Change-Id: I3c0daa1c9117eecd1566a1d52d794e92dc292fa4
CRs-Fixed: 2682570
Padma Raghunathan 5 years ago
parent
commit
d38acd0c2f
1 changed files with 3 additions and 6 deletions
  1. 3 6
      hal/wifi3.0/hal_generic_api.h

+ 3 - 6
hal/wifi3.0/hal_generic_api.h

@@ -1549,11 +1549,8 @@ hal_rx_status_get_tlv_info_generic(void *rx_tlv_hdr, void *ppduinfo,
 	}
 	}
 	case WIFIRX_MPDU_START_E:
 	case WIFIRX_MPDU_START_E:
 	{
 	{
-		uint8_t *rx_mpdu_start =
-			(uint8_t *)rx_tlv + HAL_RX_OFFSET(UNIFIED_RX_MPDU_START_0,
-					RX_MPDU_INFO_RX_MPDU_INFO_DETAILS);
-		uint32_t ppdu_id =
-				HAL_RX_GET_PPDU_ID(rx_mpdu_start);
+		uint8_t *rx_mpdu_start = (uint8_t *)rx_tlv;
+		uint32_t ppdu_id = HAL_RX_GET_PPDU_ID(rx_mpdu_start);
 		uint8_t filter_category = 0;
 		uint8_t filter_category = 0;
 
 
 		ppdu_info->nac_info.fc_valid =
 		ppdu_info->nac_info.fc_valid =
@@ -1602,7 +1599,7 @@ hal_rx_status_get_tlv_info_generic(void *rx_tlv_hdr, void *ppduinfo,
 		} else {
 		} else {
 			ppdu_info->rx_status.ppdu_len +=
 			ppdu_info->rx_status.ppdu_len +=
 				HAL_RX_GET(rx_mpdu_start, RX_MPDU_INFO_13,
 				HAL_RX_GET(rx_mpdu_start, RX_MPDU_INFO_13,
-				MPDU_LENGTH);
+					   MPDU_LENGTH);
 		}
 		}
 
 
 		filter_category =
 		filter_category =