From de4644aa732bc5867f664bdaa70d35d18e67852f Mon Sep 17 00:00:00 2001 From: Justin Newberry Date: Sat, 10 Feb 2024 17:49:45 -0500 Subject: [PATCH] camerad: assert isp started successfully (#31385) * isp assert * disable this for testing * Revert "disable this for testing" This reverts commit e65cf6d4b457babcc1a980d40079a2d6aa6540b8. * move below old-commit-hash: c9951e9d6c14b860d4b13952fca5540a91ff5813 --- system/camerad/cameras/camera_qcom2.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/system/camerad/cameras/camera_qcom2.cc b/system/camerad/cameras/camera_qcom2.cc index e17203a7dd..3191b821ac 100644 --- a/system/camerad/cameras/camera_qcom2.cc +++ b/system/camerad/cameras/camera_qcom2.cc @@ -606,6 +606,7 @@ void CameraState::camera_open(MultiCameraState *multi_cam_state_, int camera_num LOGD("start csiphy: %d", ret); ret = device_control(multi_cam_state->isp_fd, CAM_START_DEV, session_handle, isp_dev_handle); LOGD("start isp: %d", ret); + assert(ret == 0); // TODO: this is unneeded, should we be doing the start i2c in a different way? //ret = device_control(sensor_fd, CAM_START_DEV, session_handle, sensor_dev_handle);