Browse Source

qcacmn: Use HAL_RX_GET_64 to read cfr elements

Host is failing to read few cfr elements by HAL_RX_GET for
pebble as HAL_RX_GET follows 32 bit approach and pebble uses 
64-bit format and so reading 4 bytes is not enough.
Used HAL_RX_GET_64 to read those cfr elements and this change 
reads 8 bytes and then the elements are properly read.

Change-Id: If6f0b8e2a215719a1e42828764db01fe99a7e48d
CRs-Fixed: 3575631
Vaishnavi Chekuru 1 year ago
parent
commit
142ccb390a
1 changed files with 39 additions and 29 deletions
  1. 39 29
      hal/wifi3.0/qcn6432/hal_6432.c

+ 39 - 29
hal/wifi3.0/qcn6432/hal_6432.c

@@ -278,13 +278,13 @@ void hal_rx_get_bb_info_6432(void *rx_tlv, void *ppdu_info_hdl)
 	struct hal_rx_ppdu_info *ppdu_info	= ppdu_info_hdl;
 
 	ppdu_info->cfr_info.bb_captured_channel =
-		HAL_RX_GET(rx_tlv, RXPCU_PPDU_END_INFO, BB_CAPTURED_CHANNEL);
+		HAL_RX_GET_64(rx_tlv, RXPCU_PPDU_END_INFO, BB_CAPTURED_CHANNEL);
 
 	ppdu_info->cfr_info.bb_captured_timeout =
-		HAL_RX_GET(rx_tlv, RXPCU_PPDU_END_INFO, BB_CAPTURED_TIMEOUT);
+		HAL_RX_GET_64(rx_tlv, RXPCU_PPDU_END_INFO, BB_CAPTURED_TIMEOUT);
 
 	ppdu_info->cfr_info.bb_captured_reason =
-		HAL_RX_GET(rx_tlv, RXPCU_PPDU_END_INFO, BB_CAPTURED_REASON);
+		HAL_RX_GET_64(rx_tlv, RXPCU_PPDU_END_INFO, BB_CAPTURED_REASON);
 }
 
 static inline
@@ -293,55 +293,65 @@ void hal_rx_get_rtt_info_6432(void *rx_tlv, void *ppdu_info_hdl)
 	struct hal_rx_ppdu_info *ppdu_info	= ppdu_info_hdl;
 
 	ppdu_info->cfr_info.rx_location_info_valid =
-		HAL_RX_GET(rx_tlv, PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
-				RX_LOCATION_INFO_VALID);
+		HAL_RX_GET_64(rx_tlv, PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
+			      RX_LOCATION_INFO_VALID);
 
 	ppdu_info->cfr_info.rtt_che_buffer_pointer_low32 =
-		HAL_RX_GET(rx_tlv,
-				PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
-				RTT_CHE_BUFFER_POINTER_LOW32);
+		HAL_RX_GET_64(rx_tlv,
+			      PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
+			      RTT_CHE_BUFFER_POINTER_LOW32);
 
 	ppdu_info->cfr_info.rtt_che_buffer_pointer_high8 =
-		HAL_RX_GET(rx_tlv,
-				PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
-				RTT_CHE_BUFFER_POINTER_HIGH8);
+		HAL_RX_GET_64(rx_tlv,
+			      PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
+			      RTT_CHE_BUFFER_POINTER_HIGH8);
 
 	ppdu_info->cfr_info.chan_capture_status =
 		HAL_GET_RX_LOCATION_INFO_CHAN_CAPTURE_STATUS(rx_tlv);
 
 	ppdu_info->cfr_info.rx_start_ts =
-		HAL_RX_GET(rx_tlv,
-				PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
-				RX_START_TS);
+		HAL_RX_GET_64(rx_tlv,
+			      PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
+			      RX_START_TS);
 
 	ppdu_info->cfr_info.rtt_cfo_measurement = (int16_t)
-		HAL_RX_GET(rx_tlv,
-				PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
-				RTT_CFO_MEASUREMENT);
+		HAL_RX_GET_64(rx_tlv,
+			      PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
+			      RTT_CFO_MEASUREMENT);
 
 	ppdu_info->cfr_info.agc_gain_info0 =
-		HAL_RX_GET(rx_tlv,
-				PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
-				GAIN_CHAIN0);
+		HAL_RX_GET_64(rx_tlv,
+			      PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
+			      GAIN_CHAIN0);
 
 	ppdu_info->cfr_info.agc_gain_info0 |=
-		(((uint32_t)HAL_RX_GET(rx_tlv,
-				       PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
-				       GAIN_CHAIN1)) << 16);
+	(((uint32_t)HAL_RX_GET_64(rx_tlv,
+				  PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
+				  GAIN_CHAIN1)) << 16);
 
 	ppdu_info->cfr_info.agc_gain_info1 =
-		HAL_RX_GET(rx_tlv,
-				PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
-				GAIN_CHAIN2);
+		HAL_RX_GET_64(rx_tlv,
+			      PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
+			      GAIN_CHAIN2);
 
 	ppdu_info->cfr_info.agc_gain_info1 |=
-		(((uint32_t)HAL_RX_GET(rx_tlv,
-				       PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
-				       GAIN_CHAIN3)) << 16);
+	(((uint32_t)HAL_RX_GET_64(rx_tlv,
+				  PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
+				  GAIN_CHAIN3)) << 16);
 
 	ppdu_info->cfr_info.agc_gain_info2 = 0;
 
 	ppdu_info->cfr_info.agc_gain_info3 = 0;
+
+	ppdu_info->cfr_info.mcs_rate =
+	HAL_RX_GET_64(rx_tlv,
+		      PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
+		      RTT_MCS_RATE);
+
+	ppdu_info->cfr_info.gi_type =
+	HAL_RX_GET_64(rx_tlv,
+		      PHYRX_LOCATION_RX_LOCATION_INFO_DETAILS,
+		      RTT_GI_TYPE);
 }
 #endif
 #ifdef CONFIG_WORD_BASED_TLV