[libcamera-devel] [PATCH] libcamera: pipeline: ipu3: Report error when failing to configure
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Wed Jun 16 13:27:28 CEST 2021
Hi Umang,
On Wed, Jun 16, 2021 at 04:40:42PM +0530, Umang Jain wrote:
> On 6/16/21 3:26 PM, Kieran Bingham wrote:
> > If the IPA fails to configure, this can now be caught by the pipeline handler
> > but the cause may not be clear.
> >
> > If the IPA is isolated, then reports from that IPA will not be visible
> > in the libcamera logs directly.
>
> Drive by: We discussed file-logging for IPAs(isolated or otherwise). I
> wonder while at it,
> would it be worth to let pipeline-handler know how IPA is loaded (in
> isolated context
> or otherwise) by querying a flag in Proxy maybe and log that flag value
> in PH(s)?
I'd really, really prefer not making it possible for pipeline handlers
to differenciate between IPAs. If we want to log this information, we
can do so in IPAManager::createIPA() for instance.
> > Print the return error value to help identify any issue that has arisen.
> >
> > Signed-off-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
>
> Reviewed-by: Umang Jain <umang.jain at ideasonboard.com>
>
> > ---
> > src/libcamera/pipeline/ipu3/ipu3.cpp | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/src/libcamera/pipeline/ipu3/ipu3.cpp b/src/libcamera/pipeline/ipu3/ipu3.cpp
> > index 35f1e977ab19..113e70a5e692 100644
> > --- a/src/libcamera/pipeline/ipu3/ipu3.cpp
> > +++ b/src/libcamera/pipeline/ipu3/ipu3.cpp
> > @@ -644,7 +644,8 @@ int PipelineHandlerIPU3::configure(Camera *camera, CameraConfiguration *c)
> >
> > ret = data->ipa_->configure(configInfo);
> > if (ret) {
> > - LOG(IPU3, Error) << "Failed to configure IPA";
> > + LOG(IPU3, Error) << "Failed to configure IPA: "
> > + << strerror(-ret);
> > return ret;
> > }
> >
--
Regards,
Laurent Pinchart
More information about the libcamera-devel
mailing list