[libcamera-devel] [PATCH 05/12] android: camera_device: Report EXPOSURE_TIME

[Niklas Söderlund]

Hi Jacopo,

Thanks for your patch.

On 2021-01-05 20:05:15 +0100, Jacopo Mondi wrote:
> Instead of reporting a fake value in the dynamic metadata for the
> EXPOSURE_TIME control, use the value provided by the pipeline.
> 
> The metadata is only meaningfull in FULL mode.
> 
> Signed-off-by: Jacopo Mondi <jacopo at jmondi.org>

Reviewed-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>

> ---
>  src/android/camera_device.cpp | 11 ++++++-----
>  1 file changed, 6 insertions(+), 5 deletions(-)
> 
> diff --git a/src/android/camera_device.cpp b/src/android/camera_device.cpp
> index de3ead275e91..e6cee75581f2 100644
> --- a/src/android/camera_device.cpp
> +++ b/src/android/camera_device.cpp
> @@ -1871,11 +1871,6 @@ CameraDevice::getResultMetadata(Camera3RequestDescriptor *descriptor,
>  	resultMetadata->addEntry(ANDROID_SENSOR_ROLLING_SHUTTER_SKEW,
>  				 &rolling_shutter_skew, 1);
>  
> -	/* 16.6 msec */
> -	const int64_t exposure_time = 16600000;
> -	resultMetadata->addEntry(ANDROID_SENSOR_EXPOSURE_TIME,
> -				 &exposure_time, 1);
> -
>  	const uint8_t lens_shading_map_mode =
>  				ANDROID_STATISTICS_LENS_SHADING_MAP_MODE_OFF;
>  	resultMetadata->addEntry(ANDROID_STATISTICS_LENS_SHADING_MAP_MODE,
> @@ -1893,6 +1888,12 @@ CameraDevice::getResultMetadata(Camera3RequestDescriptor *descriptor,
>  					 &pipeline_depth, 1);
>  	}
>  
> +	if (metadata.contains(controls::ExposureTime)) {
> +		int32_t exposure = metadata.get(controls::ExposureTime);
> +		resultMetadata->addEntry(ANDROID_SENSOR_EXPOSURE_TIME,
> +					 &exposure, 1);
> +	}
> +
>  	/*
>  	 * Return the result metadata pack even is not valid: get() will return
>  	 * nullptr.
> -- 
> 2.29.2
> 
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel

-- 
Regards,
Niklas Söderlund