[v3] libcamera: camera_sensor_properties: Add OmniVision OV08X40 properties
diff mbox series

Message ID 2c0debc3ddb9a381e6fd2511376d4394@inspiredexperts.com
State New
Headers show
Series
  • [v3] libcamera: camera_sensor_properties: Add OmniVision OV08X40 properties
Related show

Commit Message

James Alexander Sept. 26, 2026, 9:56 p.m. UTC
The ov08x40 sensor helper is already present, but the sensor has no 
static
properties. This adds its documented 702 nm unit-cell size and 
test-pattern
mapping.

The unit-cell size comes from OmniVision's OV08X40 product information. 
Control
delays remain unspecified so the common defaults are used.

On the tested HP Spectre, setting the sensor's test-pattern control 
while
streaming at 3848x2416 showed that driver value 2 produces eight 
full-height
colour bars matching TestPatternModeColorBars after accounting for 
horizontal
orientation. Values 1, 3, and 4 remain unmapped because their libcamera
equivalents have not been established.

Signed-off-by: James Alexander <opensource@inspiredexperts.com>
---
Changes in v3:
- Correct the hardware-tested pattern mapping: value 2 produces eight
   full-height colour bars, not fading colour bars.
- Leave values 1, 3, and 4 unmapped.

Changes in v2:
- Use the documented 0.702 um unit-cell size.
- Map test-pattern value 2 to fading colour bars.
- Use the default control delays.

  src/libcamera/sensor/camera_sensor_properties.cpp | 14 ++++++++++++++
  1 file changed, 14 insertions(+)

  			.testPatternModes = {

Patch
diff mbox series

diff --git a/src/libcamera/sensor/camera_sensor_properties.cpp 
b/src/libcamera/sensor/camera_sensor_properties.cpp
index 20cfd0bc1..105c7334d 100644
--- a/src/libcamera/sensor/camera_sensor_properties.cpp
+++ b/src/libcamera/sensor/camera_sensor_properties.cpp
@@ -524,6 +524,20 @@  const CameraSensorProperties 
*CameraSensorProperties::get(const std::string &sen
  				.hblankDelay = 2
  			},
  		} },
+		{ "ov08x40", {
+			.unitCellSize = { 702, 702 },
+			.testPatternModes = {
+				{ controls::draft::TestPatternModeOff, 0 },
+				{ controls::draft::TestPatternModeColorBars, 2 },
+				/*
+				 * No corresponding test pattern mode for
+				 * 1: "Vertical Color Bar Type 1",
+				 * 3: "Vertical Color Bar Type 3",
+				 * 4: "Vertical Color Bar Type 4"
+				 */
+			},
+			.sensorDelays = { },
+		} },
  		{ "ov8858", {
  			.unitCellSize = { 1120, 1120 },