Browse Source

qcacmn: Separate WMI MGMT RX event logging

Separate WMI MGMT RX event logging from main WMI event
logging because WMI MGMT RX event is too frequent and its
over-running useful WMI control path events.

Change-Id: Iacd1576c3e133b70224e45f589f566c73637a626
CRs-Fixed: 2318021
Rajeev Kumar 6 years ago
parent
commit
183ef8819f
2 changed files with 16 additions and 12 deletions
  1. 9 7
      wmi/src/wmi_unified_non_tlv.c
  2. 7 5
      wmi/src/wmi_unified_tlv.c

+ 9 - 7
wmi/src/wmi_unified_non_tlv.c

@@ -8465,15 +8465,17 @@ static QDF_STATUS extract_smartlog_event_non_tlv
 
 static bool is_management_record_non_tlv(uint32_t cmd_id)
 {
-	if ((cmd_id == WMI_BCN_TX_CMDID) ||
-		(cmd_id == WMI_PDEV_SEND_BCN_CMDID) ||
-		(cmd_id == WMI_MGMT_TX_CMDID) ||
-		(cmd_id == WMI_GPIO_OUTPUT_CMDID) ||
-		(cmd_id == WMI_HOST_SWBA_EVENTID)) {
+	switch (cmd_id) {
+	case WMI_BCN_TX_CMDID:
+	case WMI_MGMT_TX_CMDID:
+	case WMI_MGMT_RX_EVENTID:
+	case WMI_GPIO_OUTPUT_CMDID:
+	case WMI_HOST_SWBA_EVENTID:
+	case WMI_PDEV_SEND_BCN_CMDID:
 		return true;
+	default:
+		return false;
 	}
-
-	return false;
 }
 
 static bool is_diag_event_non_tlv(uint32_t event_id)

+ 7 - 5
wmi/src/wmi_unified_tlv.c

@@ -20366,13 +20366,15 @@ static QDF_STATUS extract_peer_delete_response_event_tlv(wmi_unified_t wmi_hdl,
 
 static bool is_management_record_tlv(uint32_t cmd_id)
 {
-	if ((cmd_id == WMI_MGMT_TX_COMPLETION_EVENTID) ||
-			(cmd_id == WMI_MGMT_TX_SEND_CMDID) ||
-			(cmd_id == WMI_OFFCHAN_DATA_TX_SEND_CMDID)) {
+	switch (cmd_id) {
+	case WMI_MGMT_TX_SEND_CMDID:
+	case WMI_MGMT_TX_COMPLETION_EVENTID:
+	case WMI_OFFCHAN_DATA_TX_SEND_CMDID:
+	case WMI_MGMT_RX_EVENTID:
 		return true;
+	default:
+		return false;
 	}
-
-	return false;
 }
 
 static bool is_diag_event_tlv(uint32_t event_id)