[libcamera-devel] [PATCH v2 2/2] src: ipa: raspberrypi: Add digital gain to libcamera metadata

[Laurent Pinchart]

Hi David,

Thank you for the patch.

On Thu, Nov 26, 2020 at 02:50:05PM +0000, David Plowman wrote:
> The digital gain reported by the AGC algorithm is reported in the
> metadata that is included in completed requests.
> 
> Signed-off-by: David Plowman <david.plowman at raspberrypi.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

I'll push to the master branch shortly after the tests finish running.

> ---
>  src/ipa/raspberrypi/raspberrypi.cpp | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/src/ipa/raspberrypi/raspberrypi.cpp b/src/ipa/raspberrypi/raspberrypi.cpp
> index 9853a343..631fe7b3 100644
> --- a/src/ipa/raspberrypi/raspberrypi.cpp
> +++ b/src/ipa/raspberrypi/raspberrypi.cpp
> @@ -403,8 +403,10 @@ void IPARPi::reportMetadata()
>  	}
>  
>  	AgcStatus *agcStatus = rpiMetadata_.GetLocked<AgcStatus>("agc.status");
> -	if (agcStatus)
> +	if (agcStatus) {
>  		libcameraMetadata_.set(controls::AeLocked, agcStatus->locked);
> +		libcameraMetadata_.set(controls::DigitalGain, agcStatus->digital_gain);
> +	}
>  
>  	LuxStatus *luxStatus = rpiMetadata_.GetLocked<LuxStatus>("lux.status");
>  	if (luxStatus)

-- 
Regards,

Laurent Pinchart