commit a9ebe0a7ca090e85b3c3ef16fa4ec79ae0117e45
author Ravi Joshi <ravij@codeaurora.org> Sat Jun 17 16:43:02 2017 -0700
committer snandini <snandini@codeaurora.org> Thu Jul 06 21:55:19 2017 -0700
tree d0223126a25bd931cb5ded1caa7a0ee99b5d387b
parent 8851f4eb1c9ece960c8212f709c23978f076b2cc

qcacmn: Fix issues in monitor mode initialization

Current monitor mode implementation does not catch
initialization failures causing it to believe successful
monitor mode initialization.

Change-Id: I5c30ac22e3084871b86cf5e2f0627988ebbb7b5d
CRs-Fixed: 2071602

dp/wifi3.0/dp_rx_mon_status.c

diff --git a/dp/wifi3.0/dp_rx_mon_status.c b/dp/wifi3.0/dp_rx_mon_status.c
index df4fba1..d11fb22 100644
--- a/dp/wifi3.0/dp_rx_mon_status.c
+++ b/dp/wifi3.0/dp_rx_mon_status.c
@@ -454,6 +454,7 @@
 	struct dp_srng *rxdma_srng;
 	uint32_t rxdma_entries;
 	struct rx_desc_pool *rx_desc_pool;
+	QDF_STATUS status;
 
 	rxdma_srng = &pdev->rxdma_mon_status_ring;
 
@@ -466,15 +467,26 @@
 			"%s: Mon RX Status Pool[%d] allocation size=%d\n",
 			__func__, pdev_id, rxdma_entries);
 
-	dp_rx_desc_pool_alloc(soc, pdev_id, rxdma_entries+1, rx_desc_pool);
+	status = dp_rx_desc_pool_alloc(soc, pdev_id, rxdma_entries+1,
+			rx_desc_pool);
+	if (QDF_IS_STATUS_SUCCESS(status)) {
+		QDF_TRACE(QDF_MODULE_ID_DP, QDF_TRACE_LEVEL_ERROR,
+			"%s: dp_rx_desc_pool_alloc() failed \n", __func__);
+		return status;
+	}
 
 	QDF_TRACE(QDF_MODULE_ID_DP, QDF_TRACE_LEVEL_WARN,
 			"%s: Mon RX Status Buffers Replenish pdev_id=%d\n",
 			__func__, pdev_id);
 
-	dp_rx_mon_status_buffers_replenish(soc, pdev_id, rxdma_srng,
-		rx_desc_pool, rxdma_entries, &desc_list, &tail,
-		HAL_RX_BUF_RBM_SW3_BM);
+	status = dp_rx_mon_status_buffers_replenish(soc, pdev_id, rxdma_srng,
+			rx_desc_pool, rxdma_entries, &desc_list, &tail,
+			HAL_RX_BUF_RBM_SW3_BM);
+	if (QDF_IS_STATUS_SUCCESS(status)) {
+		QDF_TRACE(QDF_MODULE_ID_DP, QDF_TRACE_LEVEL_ERROR,
+			"%s: dp_rx_buffers_replenish() failed \n", __func__);
+		return status;
+	}
 
 	qdf_nbuf_queue_init(&pdev->rx_status_q);