qcacld-3.0: Don't call pld_collect_rddm in trigger recovery handler

RDDM can be collected during kernel panic and pld_collect_rddm is
a blocking call that will cause WLAN watchdog if the caller is from
scheduler thread. Hence revert this change and collecting FW dump
inside kernel panic handler.

CRs-Fixed: 2506844
Change-Id: I8667896e61b856fc2e03e1c8278a6451b6a04c24

core/cds/src/cds_api.c

@@ -1785,7 +1785,6 @@ static void cds_trigger_recovery_handler(const char *func, const uint32_t line)
 	QDF_STATUS status;
 	qdf_runtime_lock_t rtl;
 	qdf_device_t qdf;
-	int ret = 0;
 
 	/* NOTE! This code path is delicate! Think very carefully before
 	 * modifying the content or order of the following. Please review any
@@ -1813,12 +1812,6 @@ static void cds_trigger_recovery_handler(const char *func, const uint32_t line)
 		return;
 	}
 
-	if (!in_interrupt() && !irqs_disabled()) {
-		ret = pld_collect_rddm(qdf->dev);
-		if (ret < 0 && ret != -EOPNOTSUPP)
-			QDF_DEBUG_PANIC("Fail to collect FW ramdump %d", ret);
-	}
-
 	/* if *wlan* recovery is disabled, crash here for debugging */
 	if (!cds_is_self_recovery_enabled()) {
 		QDF_DEBUG_PANIC("WLAN recovery is not enabled (via %s:%d)",