qcacld-3.0: Deconfigure CDS before disabling scheduler thread

On direct link datapath init failure, CDS is
deconfigured after scheduler thread is disabled.
Umac_stop is invoked as part of hdd_deconfigure_cds
which posts a message to scheduler thread and since
scheduler thread is already disabled, the message
posting fails which eventually results in timer leaks.

Fix is to deconfigure CDS before disabling scheduler
thread.

Change-Id: If34cae802ff65ebba67adb6a06833e64da3efd45
CRs-Fixed: 3551233

core/hdd/src/wlan_hdd_main.c

@@ -4769,7 +4769,6 @@ int hdd_wlan_start_modules(struct hdd_context *hdd_ctx, bool reinit)
 	void *hif_ctx;
 	struct target_psoc_info *tgt_hdl;
 	unsigned long thermal_state = 0;
-	bool is_sched_disabled = false;
 	uint8_t index = 0;
 	struct dev_set_param setparam[MAX_PDEV_PRE_ENABLE_PARAMS] = {};
 
@@ -5056,16 +5055,15 @@ int hdd_wlan_start_modules(struct hdd_context *hdd_ctx, bool reinit)
 	hdd_exit();
 
 	return 0;
-/*
- * Disable scheduler 1st so that scheduler thread doesn't send messages to fw
- * in parallel to the cleanup
- */
+
 deconfigure_cds:
-	is_sched_disabled = !dispatcher_disable();
 	hdd_deconfigure_cds(hdd_ctx);
 sched_disable:
-	if (!is_sched_disabled)
-		dispatcher_disable();
+	/*
+	 * Disable scheduler 1st so that scheduler thread doesn't send messages
+	 * to fw in parallel to the cleanup
+	 */
+	dispatcher_disable();
 	hdd_destroy_sysfs_files();
 	cds_post_disable();
 unregister_notifiers: