[PATCH] libcamera: camera_sensor_properties: ov5675: Set correct delays

Kieran Bingham kieran.bingham at ideasonboard.com
Thu Dec 19 01:59:58 CET 2024


The OV5675 uses different delays for gain and exposure than are configured
in the default sensorDelays utilised by the CameraSensorLegacy.

Empirical testing using a Lenovo X13s shows that the exposure delay is
only a single frame, and the current setting of 2 frame delay produces
exceedingly frequent oscillations in the image exposure.

Update the OV5675 sensor delays table accordingly.

Signed-off-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
---
 src/libcamera/sensor/camera_sensor_properties.cpp | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/libcamera/sensor/camera_sensor_properties.cpp b/src/libcamera/sensor/camera_sensor_properties.cpp
index bd1fc86977ce..813878386a35 100644
--- a/src/libcamera/sensor/camera_sensor_properties.cpp
+++ b/src/libcamera/sensor/camera_sensor_properties.cpp
@@ -360,7 +360,12 @@ const CameraSensorProperties *CameraSensorProperties::get(const std::string &sen
 				{ controls::draft::TestPatternModeOff, 0 },
 				{ controls::draft::TestPatternModeColorBars, 1 },
 			},
-			.sensorDelays = { },
+			.sensorDelays = {
+				.exposureDelay = 1,
+				.gainDelay = 1,
+				.vblankDelay = 2,
+				.hblankDelay = 2
+			    },
 		} },
 		{ "ov5693", {
 			.unitCellSize = { 1400, 1400 },
-- 
2.47.1



More information about the libcamera-devel mailing list