<div dir="ltr"><div dir="ltr">Hi Kieran, thank you for the patch.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jun 16, 2021 at 6:56 PM Kieran Bingham <<a href="mailto:kieran.bingham@ideasonboard.com">kieran.bingham@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">If the IPA fails to configure, this can now be caught by the pipeline handler<br>
but the cause may not be clear.<br>
<br>
If the IPA is isolated, then reports from that IPA will not be visible<br>
in the libcamera logs directly.<br>
<br>
Print the return error value to help identify any issue that has arisen.<br>
<br>
Signed-off-by: Kieran Bingham <<a href="mailto:kieran.bingham@ideasonboard.com" target="_blank">kieran.bingham@ideasonboard.com</a>><br></blockquote><div><br></div><div>Reviewed-by: Hirokazu Honda <<a href="mailto:hiroh@chromium.org">hiroh@chromium.org</a>></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
---<br>
 src/libcamera/pipeline/ipu3/ipu3.cpp | 3 ++-<br>
 1 file changed, 2 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/src/libcamera/pipeline/ipu3/ipu3.cpp b/src/libcamera/pipeline/ipu3/ipu3.cpp<br>
index 35f1e977ab19..113e70a5e692 100644<br>
--- a/src/libcamera/pipeline/ipu3/ipu3.cpp<br>
+++ b/src/libcamera/pipeline/ipu3/ipu3.cpp<br>
@@ -644,7 +644,8 @@ int PipelineHandlerIPU3::configure(Camera *camera, CameraConfiguration *c)<br>
<br>
        ret = data->ipa_->configure(configInfo);<br>
        if (ret) {<br>
-               LOG(IPU3, Error) << "Failed to configure IPA";<br>
+               LOG(IPU3, Error) << "Failed to configure IPA: "<br>
+                                << strerror(-ret);<br>
                return ret;<br>
        }<br>
<br>
-- <br>
2.30.2<br>
<br>
</blockquote></div></div>