[PATCH v2 0/3] Fix misleading error messages

[Milan Zamazal]

Milan Zamazal <mzamazal at redhat.com> writes:

> At least with software ISP, "errors" that have actually no influence on
> regular libcamera operation may be reported.  This is confusing for
> users, especially in cases when there are other, real errors.  Let's improve the given log messages a bit.
>
> Changes in v2:
> - The order of the patches swapped, to put the easier change first.
> - An empty line, as requested by Laurent, added.
> - Added a trivial patch to unify the IPAProxy::configurationFile argument
>   name in the declaration and definition.
> - Added a new optional argument to IPAProxy::configurationFile for a
>   fallback file, which solves the error/warning problem there.

OK or is there a better suggestion?

> Milan Zamazal (3):
>   libcamera: simple: Log a missing sensor in a better way
>   libcamera: ipa_proxy: Unify configurationFile argument name
>   libcamera: ipa_proxy: Report a missing configuration as a warning
>
>  include/libcamera/internal/ipa_proxy.h      |  4 +++-
>  src/libcamera/ipa_proxy.cpp                 | 22 +++++++++++++++++----
>  src/libcamera/pipeline/ipu3/ipu3.cpp        |  5 ++---
>  src/libcamera/pipeline/rkisp1/rkisp1.cpp    |  9 ++-------
>  src/libcamera/pipeline/simple/simple.cpp    |  4 +++-
>  src/libcamera/software_isp/software_isp.cpp |  5 ++---
>  6 files changed, 30 insertions(+), 19 deletions(-)