From 832107376351efe1c9f6506bfd01bacb94bc52a2 Mon Sep 17 00:00:00 2001 From: bings Date: Fri, 3 Nov 2017 09:09:52 +0800 Subject: [PATCH] qcacld-3.0: Do not destroy rx_buff_list_lock if it is not used When CONFIG_HL_SUPPORT is defined, htt_rx_dbg_rxbuf_init is not called, and rx_buff_list_lock is not initialized. rx_buff_list_lock should not be destroyed if it is not initialized. Do not call htt_rx_dbg_rxbuf_deinit when CONFIG_HL_SUPPORT is defined. Change-Id: I9d64313efd5a9978b3fb953ac5d7ffb466abe010 CRs-Fixed: 2134345 --- core/dp/htt/htt.c | 1 - core/dp/htt/htt_rx.c | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/core/dp/htt/htt.c b/core/dp/htt/htt.c index dbe9363ae0f8..5839ac977923 100644 --- a/core/dp/htt/htt.c +++ b/core/dp/htt/htt.c @@ -666,7 +666,6 @@ void htt_detach(htt_pdev_handle pdev) HTT_TX_MUTEX_DESTROY(&pdev->credit_mutex); HTT_TX_MUTEX_DESTROY(&pdev->htt_tx_mutex); HTT_TX_NBUF_QUEUE_MUTEX_DESTROY(pdev); - htt_rx_dbg_rxbuf_deinit(pdev); } /** diff --git a/core/dp/htt/htt_rx.c b/core/dp/htt/htt_rx.c index 715e16a20972..f717d39594ec 100644 --- a/core/dp/htt/htt_rx.c +++ b/core/dp/htt/htt_rx.c @@ -689,6 +689,7 @@ void htt_rx_detach(struct htt_pdev_t *pdev) { qdf_timer_stop(&pdev->rx_ring.refill_retry_timer); qdf_timer_free(&pdev->rx_ring.refill_retry_timer); + htt_rx_dbg_rxbuf_deinit(pdev); if (pdev->cfg.is_full_reorder_offload) { qdf_mem_free_consistent(pdev->osdev, pdev->osdev->dev,