From 1aadfe80e81403d76bba0eaed3437ff330584087 Mon Sep 17 00:00:00 2001 From: Vishalsingh Hajeri Date: Wed, 9 Oct 2019 17:21:41 -0700 Subject: [PATCH] msm: camera: isp: Remove redundant use of spin lock Interrupts in activated state machine are already protected with spin_lock for context. No need to reuse the spin_lock in interrupt handling routine. CRs-Fixed: 2538876 Change-Id: Id86e7dc3419861cb8bce143c56923126a3dc3d3e Signed-off-by: Vishalsingh Hajeri --- drivers/cam_isp/cam_isp_context.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/drivers/cam_isp/cam_isp_context.c b/drivers/cam_isp/cam_isp_context.c index 3ffb623f0d6d..53521e79b388 100644 --- a/drivers/cam_isp/cam_isp_context.c +++ b/drivers/cam_isp/cam_isp_context.c @@ -810,21 +810,17 @@ static int __cam_isp_ctx_notify_sof_in_activated_state( return rc; } - spin_lock_bh(&ctx->lock); req = list_first_entry(&ctx->active_req_list, struct cam_ctx_request, list); req_isp = (struct cam_isp_ctx_req *) req->req_priv; - spin_unlock_bh(&ctx->lock); if (ctx_isp->bubble_frame_cnt >= 1 && req_isp->bubble_detected) { req_isp->num_acked = 0; ctx_isp->bubble_frame_cnt = 0; req_isp->bubble_detected = false; - spin_lock_bh(&ctx->lock); list_del_init(&req->list); list_add(&req->list, &ctx->pending_req_list); - spin_unlock_bh(&ctx->lock); atomic_set(&ctx_isp->process_bubble, 0); ctx_isp->active_req_cnt--; CAM_DBG(CAM_REQ,