Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit da36cd6f authored by Ravikishore Pampana's avatar Ravikishore Pampana Committed by Gerrit - the friendly Code Review server
Browse files

msm: camera: Return IRQ_NONE from irq handlers



IRQ_NONE need to return even if irq handler get the invalid
parameters or encounter error condition.
If IRQ_NONE handled value not return from irq handler, same
irq comes again.

Change-Id: I94f4ac9935a2d611734be8f669d5d43388d63ab4
Signed-off-by: default avatarRavikishore Pampana <rpampana@codeaurora.org>
parent 6ac7964f
Loading
Loading
Loading
Loading
+2 −2
Original line number Diff line number Diff line
@@ -532,7 +532,7 @@ irqreturn_t cam_fd_hw_irq(int irq_num, void *data)

	if (!fd_hw) {
		CAM_ERR(CAM_FD, "Invalid data in IRQ callback");
		return -EINVAL;
		return IRQ_NONE;
	}

	fd_core = (struct cam_fd_core *) fd_hw->core_info;
@@ -570,7 +570,7 @@ irqreturn_t cam_fd_hw_irq(int irq_num, void *data)
		CAM_ERR(CAM_FD,
			"Invalid number of IRQs, value=0x%x, num_irqs=%d",
			reg_value, num_irqs);
		return -EINVAL;
		return IRQ_NONE;
	}

	trace_cam_irq_activated("FD", irq_type);
+2 −2
Original line number Diff line number Diff line
@@ -1118,7 +1118,7 @@ irqreturn_t cam_lrme_hw_irq(int irq_num, void *data)

	if (!data) {
		CAM_ERR(CAM_LRME, "Invalid data in IRQ callback");
		return -EINVAL;
		return IRQ_NONE;
	}

	lrme_hw = (struct cam_hw_info *)data;
@@ -1179,7 +1179,7 @@ irqreturn_t cam_lrme_hw_irq(int irq_num, void *data)
		task = cam_req_mgr_workq_get_task(lrme_core->work);
		if (!task) {
			CAM_ERR(CAM_LRME, "no empty task available");
			return -ENOMEM;
			return IRQ_NONE;
		}
		work_data = (struct cam_lrme_hw_work_data *)task->payload;
		work_data->top_irq_status = top_irq_status;
+1 −1
Original line number Diff line number Diff line
@@ -177,7 +177,7 @@ irqreturn_t cam_csiphy_irq(int irq_num, void *data)

	if (!csiphy_dev) {
		CAM_ERR(CAM_CSIPHY, "Invalid Args");
		return -EINVAL;
		return IRQ_NONE;
	}

	soc_info = &csiphy_dev->soc_info;