Browse Source

qcacmn: cck,nss and preamble update fix

cck rates were getting updated for ofdm rates
and nss index were getting wrongly incremented
fixed the same

Corrected sgi and preamble too

Change-Id: I2dabb685de6e0bd4ba5094845cde228b3c32e2a9
Anish Nataraj 7 years ago
parent
commit
072d897705
3 changed files with 19 additions and 19 deletions
  1. 18 17
      dp/wifi3.0/dp_main.c
  2. 1 0
      dp/wifi3.0/dp_rx_mon_status.c
  3. 0 2
      hal/wifi3.0/hal_api_mon.h

+ 18 - 17
dp/wifi3.0/dp_main.c

@@ -125,21 +125,7 @@ struct dp_rate_debug {
 #define MCS_INVALID 0
 
 static const struct dp_rate_debug dp_rate_string[DOT11_MAX][MAX_MCS] = {
-	{
-		{"CCK 11 Mbps Long  ", MCS_VALID},
-		{"CCK 5.5 Mbps Long ", MCS_VALID},
-		{"CCK 2 Mbps Long   ", MCS_VALID},
-		{"CCK 1 Mbps Long   ", MCS_VALID},
-		{"CCK 11 Mbps Short ", MCS_VALID},
-		{"CCK 5.5 Mbps Short", MCS_VALID},
-		{"CCK 2 Mbps Short  ", MCS_VALID},
-		{"INVALID ", MCS_INVALID},
-		{"INVALID ", MCS_INVALID},
-		{"INVALID ", MCS_INVALID},
-		{"INVALID ", MCS_INVALID},
-		{"INVALID ", MCS_INVALID},
-		{"INVALID ", MCS_VALID},
-	},
+
 	{
 		{"OFDM 48 Mbps", MCS_VALID},
 		{"OFDM 24 Mbps", MCS_VALID},
@@ -155,6 +141,21 @@ static const struct dp_rate_debug dp_rate_string[DOT11_MAX][MAX_MCS] = {
 		{"INVALID ", MCS_INVALID},
 		{"INVALID ", MCS_VALID},
 	},
+	{
+		{"CCK 11 Mbps Long  ", MCS_VALID},
+		{"CCK 5.5 Mbps Long ", MCS_VALID},
+		{"CCK 2 Mbps Long   ", MCS_VALID},
+		{"CCK 1 Mbps Long   ", MCS_VALID},
+		{"CCK 11 Mbps Short ", MCS_VALID},
+		{"CCK 5.5 Mbps Short", MCS_VALID},
+		{"CCK 2 Mbps Short  ", MCS_VALID},
+		{"INVALID ", MCS_INVALID},
+		{"INVALID ", MCS_INVALID},
+		{"INVALID ", MCS_INVALID},
+		{"INVALID ", MCS_INVALID},
+		{"INVALID ", MCS_INVALID},
+		{"INVALID ", MCS_VALID},
+	},
 	{
 		{"HT MCS 0 (BPSK 1/2)  ", MCS_VALID},
 		{"HT MCS 1 (QPSK 1/2)  ", MCS_VALID},
@@ -4648,7 +4649,7 @@ dp_print_rx_rates(struct dp_vdev *vdev)
 		index += qdf_snprint(&nss[index], DP_NSS_LENGTH - index,
 				" %d", pdev->stats.rx.nss[i]);
 	}
-	DP_PRINT_STATS("NSS(0-7) = %s",
+	DP_PRINT_STATS("NSS(1-8) = %s",
 			nss);
 
 	DP_PRINT_STATS("SGI ="
@@ -4930,7 +4931,7 @@ static inline void dp_print_peer_stats(struct dp_peer *peer)
 		index += qdf_snprint(&nss[index], DP_NSS_LENGTH - index,
 				" %d", peer->stats.rx.nss[i]);
 	}
-	DP_PRINT_STATS("NSS(0-7) = %s",
+	DP_PRINT_STATS("NSS(1-8) = %s",
 			nss);
 
 	DP_PRINT_STATS("Aggregation:");

+ 1 - 0
dp/wifi3.0/dp_rx_mon_status.c

@@ -79,6 +79,7 @@ dp_rx_populate_cdp_indication_ppdu(struct dp_soc *soc,
 	cdp_rx_ppdu->other_msdu_count = ppdu_info->rx_status.other_msdu_count;
 	cdp_rx_ppdu->u.nss = ppdu_info->rx_status.nss;
 	cdp_rx_ppdu->u.mcs = ppdu_info->rx_status.mcs;
+	cdp_rx_ppdu->u.gi = ppdu_info->rx_status.sgi;
 	cdp_rx_ppdu->u.preamble = ppdu_info->rx_status.preamble_type;
 	cdp_rx_ppdu->rssi = ppdu_info->rx_status.rssi_comb;
 	cdp_rx_ppdu->timestamp = ppdu_info->com_info.ppdu_timestamp;

+ 0 - 2
hal/wifi3.0/hal_api_mon.h

@@ -894,8 +894,6 @@ hal_rx_status_get_tlv_info(void *rx_tlv, struct hal_rx_ppdu_info *ppdu_info)
 #else
 			PHYRX_RSSI_LEGACY_0, RECEIVE_BANDWIDTH);
 #endif
-		ppdu_info->rx_status.preamble_type = HAL_RX_GET(rx_tlv,
-			PHYRX_RSSI_LEGACY_0, RECEPTION_TYPE);
 		ppdu_info->rx_status.he_re = 0;
 
 		value = HAL_RX_GET(rssi_info_tlv,