video: driver: fix client_lock acquire sequence

Updated client_lock acquire sequence and cleaned
up timer list.

Change-Id: I413f31c8f4622f152598b08c9e107c77c03525cc
Signed-off-by: Govindaraj Rajagopal <quic_grajagop@quicinc.com>

driver/vidc/src/msm_vidc_driver.c

@@ -5727,6 +5727,7 @@ void msm_vidc_destroy_buffers(struct msm_vidc_inst *inst)
 	list_for_each_entry_safe(timer, dummy_timer, &inst->input_timer_list, list) {
 		i_vpr_e(inst, "%s: removing input_timer %lld\n",
 			__func__, timer->time_us);
+		list_del(&timer->list);
 		msm_memory_pool_free(inst, timer);
 	}
 

driver/vidc/src/msm_vidc_v4l2.c

@@ -381,8 +381,8 @@ int msm_v4l2_create_bufs(struct file *filp, void *fh,
 		return -EINVAL;
 	}
 
-	inst_lock(inst, __func__);
 	client_lock(inst, __func__);
+	inst_lock(inst, __func__);
 	rc = msm_vidc_create_bufs((void *)inst, b);
 	if (rc)
 		goto unlock;