[libcamera-devel] [PATCH v4 11/14] ipa: ipu3: Send color temperature in the metadata

[Paul Elder]

Hi Jean-Michel,

On Thu, Nov 11, 2021 at 03:09:25PM +0100, Jean-Michel Hautbois wrote:
> Now that the color temperature is updated per-frame, use the value and
> set the corresponding controls::ColourTemperature.
> 
> Signed-off-by: Jean-Michel Hautbois <jeanmichel.hautbois at ideasonboard.com>
> Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

Reviewed-by: Paul Elder <paul.elder at ideasonboard.com>

> ---
>  src/ipa/ipu3/ipu3.cpp | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/src/ipa/ipu3/ipu3.cpp b/src/ipa/ipu3/ipu3.cpp
> index 38e86e58..d3195de6 100644
> --- a/src/ipa/ipu3/ipu3.cpp
> +++ b/src/ipa/ipu3/ipu3.cpp
> @@ -631,6 +631,16 @@ void IPAIPU3::parseStatistics(unsigned int frame,
>  				(sensorInfo_.pixelRate / 1e6);
>  	ctrls.set(controls::FrameDuration, frameDuration);
>  
> +	ctrls.set(controls::ColourTemperature, context_.frameContext.awb.temperatureK);
> +
> +	/*
> +	 * \todo The Metadata provides a path to getting extended data
> +	 * out to the application. Further data such as a simplifed Histogram
> +	 * might have value to be exposed, however such data may be
> +	 * difficult to report in a generically parsable way and we
> +	 * likely want to avoid putting platform specific metadata in.
> +	 */
> +
>  	IPU3Action op;
>  	op.op = ActionMetadataReady;
>  	op.controls = ctrls;
> -- 
> 2.32.0
>