Mirror/libcamera
1
0
Fork 0
mirror of https://git.libcamera.org/libcamera/libcamera.git synced 2025-07-16 17:05:08 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions

libcamera: ipu3: Initialize controls using sensor resolution

Browse source 
The controls' limits initialized by the IPU3 pipeline handler depend
on the sensor configuration. In order to compute controls using a known
state apply to the sensor a configuration equal to its own resolution.

Move the \todo note regarding the controls' limits dependency on the
sensor configuration at the beginning of the function and remove the
other redundant ones in the function body.

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Jacopo Mondi <jacopo@jmondi.org>
This commit is contained in:
Jacopo Mondi 2021-02-18 20:43:18 +01:00
parent 45ebe9a209
commit cbd4617a09
1 changed files with 19 additions and 20 deletions
Download patch file Download diff file Expand all files Collapse all files

39
src/libcamera/pipeline/ipu3/ipu3.cpp
View file

@ -868,10 +868,23 @@ bool PipelineHandlerIPU3::match(DeviceEnumerator *enumerator)
*/ */
int PipelineHandlerIPU3::initControls(IPU3CameraData *data) int PipelineHandlerIPU3::initControls(IPU3CameraData *data)
{ {
/*
* \todo The constrols intialized here depend on sensor configuration
* and their limits should be updated once the configuration gets
* changed.
*
* Initialize the sensor using its resolution and compute the control
* limits.
*/
CameraSensor *sensor = data->cio2_.sensor(); CameraSensor *sensor = data->cio2_.sensor();
CameraSensorInfo sensorInfo{}; V4L2SubdeviceFormat sensorFormat = {};
sensorFormat.size = sensor->resolution();
int ret = sensor->setFormat(&sensorFormat);
if (ret)
return ret;
int ret = sensor->sensorInfo(&sensorInfo); CameraSensorInfo sensorInfo{};
ret = sensor->sensorInfo(&sensorInfo);
if (ret) if (ret)
return ret; return ret;
@ -880,7 +893,6 @@ int PipelineHandlerIPU3::initControls(IPU3CameraData *data)
/* /*
* Compute exposure time limits. * Compute exposure time limits.
* *
* \todo The exposure limits depend on the sensor configuration.
* Initialize the control using the line length and pixel rate of the * Initialize the control using the line length and pixel rate of the
* current configuration converted to microseconds. Use the * current configuration converted to microseconds. Use the
* V4L2_CID_EXPOSURE control to get exposure min, max and default and * V4L2_CID_EXPOSURE control to get exposure min, max and default and
@ -906,25 +918,12 @@ int PipelineHandlerIPU3::initControls(IPU3CameraData *data)
/* /*
* Compute the scaler crop limits. * Compute the scaler crop limits.
* *
* \todo The scaler crop limits depend on the sensor configuration. It * Initialize the control use the 'Viewfinder' configuration (1280x720)
* should be updated when a new configuration is applied. To initialize * as the pipeline output resolution and the full sensor size as input
* the control use the 'Viewfinder' configuration (1280x720) as the * frame (see the todo note in the validate() function about the usage
* pipeline output resolution and the full sensor size as input frame * of the sensor's full frame as ImgU input).
* (see the todo note in the validate() function about the usage of the
* sensor's full frame as ImgU input).
*/ */
/* Re-fetch the sensor info updated to use the largest resolution. */
V4L2SubdeviceFormat sensorFormat = {};
sensorFormat.size = sensor->resolution();
ret = sensor->setFormat(&sensorFormat);
if (ret)
return ret;
ret = sensor->sensorInfo(&sensorInfo);
if (ret)
return ret;
/* /*
* The maximum scaler crop rectangle is the analogue crop used to * The maximum scaler crop rectangle is the analogue crop used to
* produce the maximum frame size. * produce the maximum frame size.