[libcamera-devel] [PATCH] ipa: raspberrypi: Return controls::FrameDuration from the IPA
David Plowman
david.plowman at raspberrypi.com
Wed Jul 28 10:22:45 CEST 2021
Hi Laurent
Sorry, this one must have slipped past me! Happy to merge, also with
Reviewed-by: David Plowman <david.plowman at raspberrypi.com>
Thanks!
Davod
On Wed, 28 Jul 2021 at 07:55, Laurent Pinchart
<laurent.pinchart at ideasonboard.com> wrote:
>
> 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
More information about the libcamera-devel
mailing list