From fff07541e4cfa609a8b6814a0e327b8bb3f5430d Mon Sep 17 00:00:00 2001 From: Vulupala Shashank Reddy Date: Mon, 23 Aug 2021 11:31:32 +0530 Subject: [PATCH] qcacld-3.0: Unsubscribe in reverse order of subscription Ideally we should deinit in reverse order of init so we should Unsubscribe in reverse order of subscription in packet capture component. Change-Id: Icb62a14c2d1fd20073f6924d03d523b7871c62d6 CRs-Fixed: 3019862 --- .../core/src/wlan_pkt_capture_main.c | 28 +++++++++---------- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/components/pkt_capture/core/src/wlan_pkt_capture_main.c b/components/pkt_capture/core/src/wlan_pkt_capture_main.c index 25f3f525b004..3e2703e2043d 100644 --- a/components/pkt_capture/core/src/wlan_pkt_capture_main.c +++ b/components/pkt_capture/core/src/wlan_pkt_capture_main.c @@ -114,25 +114,25 @@ static void pkt_capture_wdi_event_unsubscribe(struct wlan_objmgr_psoc *psoc) void *soc = cds_get_context(QDF_MODULE_ID_SOC); uint8_t pdev_id = WMI_PDEV_ID_SOC; - /* unsubscribing for tx data packets */ - cdp_wdi_event_unsub(soc, pdev_id, &PKT_CAPTURE_TX_SUBSCRIBER, - WDI_EVENT_PKT_CAPTURE_TX_DATA); - - /* unsubscribing for rx data packets */ - cdp_wdi_event_unsub(soc, pdev_id, &PKT_CAPTURE_RX_SUBSCRIBER, - WDI_EVENT_PKT_CAPTURE_RX_DATA); - - /* unsubscribe for rx data no peer packets */ - cdp_wdi_event_sub(soc, pdev_id, &PKT_CAPTURE_RX_NO_PEER_SUBSCRIBER, - WDI_EVENT_PKT_CAPTURE_RX_DATA_NO_PEER); + /* unsubscribe ppdu smu stats */ + cdp_wdi_event_unsub(soc, pdev_id, &PKT_CAPTURE_PPDU_STATS_SUBSCRIBER, + WDI_EVENT_PKT_CAPTURE_PPDU_STATS); /* unsubscribing for offload tx data packets */ cdp_wdi_event_unsub(soc, pdev_id, &PKT_CAPTURE_OFFLOAD_TX_SUBSCRIBER, WDI_EVENT_PKT_CAPTURE_OFFLOAD_TX_DATA); - /* unsubscribe ppdu smu stats */ - cdp_wdi_event_unsub(soc, pdev_id, &PKT_CAPTURE_PPDU_STATS_SUBSCRIBER, - WDI_EVENT_PKT_CAPTURE_PPDU_STATS); + /* unsubscribe for rx data no peer packets */ + cdp_wdi_event_sub(soc, pdev_id, &PKT_CAPTURE_RX_NO_PEER_SUBSCRIBER, + WDI_EVENT_PKT_CAPTURE_RX_DATA_NO_PEER); + + /* unsubscribing for rx data packets */ + cdp_wdi_event_unsub(soc, pdev_id, &PKT_CAPTURE_RX_SUBSCRIBER, + WDI_EVENT_PKT_CAPTURE_RX_DATA); + + /* unsubscribing for tx data packets */ + cdp_wdi_event_unsub(soc, pdev_id, &PKT_CAPTURE_TX_SUBSCRIBER, + WDI_EVENT_PKT_CAPTURE_TX_DATA); } enum pkt_capture_mode