[libcamera-devel] [PATCH 10/22] ipu3: Move delayedControls reset after IPA is started

[Hanlin Chen]

Thanks for the patch Jean-Michel,
On Mon, Nov 8, 2021 at 9:49 PM Kieran Bingham
<kieran.bingham at ideasonboard.com> wrote:
>
> Quoting Jean-Michel Hautbois (2021-11-08 13:13:38)
> > We can call configure multiple times and we don't need to reset
> > delayedControls for each of the configure calls. Reset it after the IPA
> > is started.
> >
> > Signed-off-by: Jean-Michel Hautbois <jeanmichel.hautbois at ideasonboard.com>
> > ---
> >  src/libcamera/pipeline/ipu3/ipu3.cpp | 8 ++++++--
> >  1 file changed, 6 insertions(+), 2 deletions(-)
> >
> > diff --git a/src/libcamera/pipeline/ipu3/ipu3.cpp b/src/libcamera/pipeline/ipu3/ipu3.cpp
> > index 6a7f5b9a..3fcfa777 100644
> > --- a/src/libcamera/pipeline/ipu3/ipu3.cpp
> > +++ b/src/libcamera/pipeline/ipu3/ipu3.cpp
> > @@ -667,8 +667,6 @@ int PipelineHandlerIPU3::configure(Camera *camera, CameraConfiguration *c)
> >                 return ret;
> >         }
> >
> > -       data->delayedCtrls_->reset();
> > -
> >         return updateControls(data);
> >  }
> >
> > @@ -769,6 +767,12 @@ int PipelineHandlerIPU3::start(Camera *camera, [[maybe_unused]] const ControlLis
> >         if (ret)
> >                 goto error;
> >
> > +       /*
> > +        * Reset the delayed controls with the gain and exposure values set by
> > +        * the IPA.
> > +        */
> > +       data->delayedCtrls_->reset();
> > +
>
> Should this be squashed into 2/22?
>
> Han-Lin - Does this affect your use case at all? (I suspect it's ok)
I think it should be okay as long as we reset it before the first
frame queuing into the sensor node.
> --
> Kieran
>
>
> >         /*
> >          * Start the ImgU video devices, buffers will be queued to the
> >          * ImgU output and viewfinder when requests will be queued.
> > --
> > 2.32.0
> >