From 5e1f0d8b54ab27fdbf3dbc202af1b63bf4eb10da Mon Sep 17 00:00:00 2001 From: Laurent Pinchart Date: Sun, 31 Jan 2021 19:35:44 +0200 Subject: libcamera: camera_sensor: Restrict sensor info to raw sensors YUV sensors don't provide the necessary information to fill CameraSensorInfo, as they include an ISP and provide a higher-level API that doesn't always expose low-level information. The CameraSensor class makes low-level V4L2 controls mandatory for all sensors, which prevents usage of YUV sensors with the simple pipeline handler. Make CameraSensor::sensorInfo() available for raw sensors only. This won't introduce any regression in pipeline handlers that currently use the sensorInfo() function as they all operate with raw sensors, and won't be a limitation for the simple pipeline handler as well as it doesn't use sensor info. If part of the sensor info (such as the active pixel array size for instance) becomes useful to expose for YUV sensors in the future, the sensorInfo() function can be extended to report that information only and skip data that is only available for raw sensors. Signed-off-by: Laurent Pinchart Reviewed-by: Jacopo Mondi --- Documentation/sensor_driver_requirements.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'Documentation') diff --git a/Documentation/sensor_driver_requirements.rst b/Documentation/sensor_driver_requirements.rst index 590797f0..17841ddd 100644 --- a/Documentation/sensor_driver_requirements.rst +++ b/Documentation/sensor_driver_requirements.rst @@ -22,7 +22,7 @@ Mandatory Requirements The sensor driver is assumed to be fully compliant with the V4L2 specification. -The sensor driver shall support the following V4L2 controls: +For RAW sensors, the sensor driver shall support the following V4L2 controls: * `V4L2_CID_EXPOSURE`_ * `V4L2_CID_HBLANK`_ -- cgit v1.2.1