[libcamera-devel] [PATCH] ipa: raspberrypi: Return controls::FrameDuration from the IPA

[Laurent Pinchart]

Hi Naush,

Thank you for the patch.

On Thu, Jul 22, 2021 at 12:57:42PM +0100, Naushir Patuck wrote:
> Return controls::FrameDuration through the per-frame Request metadata. The
> frame duration is obtained by either the value in DelayedControls, or
> (preferably) the value parsed from the embedded data buffer.
> 
> Signed-off-by: Naushir Patuck <naush at raspberrypi.com>

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

David, would you like to review this, or should I merge it ?

> ---
>  src/ipa/raspberrypi/raspberrypi.cpp | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/src/ipa/raspberrypi/raspberrypi.cpp b/src/ipa/raspberrypi/raspberrypi.cpp
> index 5cd333049807..b08602f474af 100644
> --- a/src/ipa/raspberrypi/raspberrypi.cpp
> +++ b/src/ipa/raspberrypi/raspberrypi.cpp
> @@ -465,6 +465,8 @@ void IPARPi::reportMetadata()
>  		libcameraMetadata_.set(controls::ExposureTime,
>  				       deviceStatus->shutter_speed.get<std::micro>());
>  		libcameraMetadata_.set(controls::AnalogueGain, deviceStatus->analogue_gain);
> +		libcameraMetadata_.set(controls::FrameDuration,
> +				       helper_->Exposure(deviceStatus->frame_length).get<std::micro>());
>  	}
>  
>  	AgcStatus *agcStatus = rpiMetadata_.GetLocked<AgcStatus>("agc.status");

-- 
Regards,

Laurent Pinchart