summaryrefslogtreecommitdiff
path: root/src/apps/common/options.h
diff options
context:
space:
mode:
authorMilan Zamazal <mzamazal@redhat.com>2024-07-31 20:36:16 +0200
committerKieran Bingham <kieran.bingham@ideasonboard.com>2024-08-01 08:50:36 +0100
commit6c17996879d0f01cf92fc77431d70723a20be70b (patch)
treed82de9b18eaac037ab7bcf2201ef9da4b72d4641 /src/apps/common/options.h
parent58f83d70ebf3ae2b98a9ade8e173c19f555a890a (diff)
libcamera: ipa_proxy: Report a missing configuration as a warning
When the configuration file for an IPA module is missing, it is reported as an error in the log, for example: ERROR IPAProxy ipa_proxy.cpp:149 Configuration file 'imx219.yaml' not found for IPA module 'simple' This is misleading because several pipelines use uncalibrated.yaml in such a case and can continue working. And in case of software ISP, there is currently no other configuration file so the error is always reported. On the other hand, in some other cases the presence of the configuration file is required and it is an error if it is missing. Let's introduce a new optional argument to IPAProxy::configurationFile that specifies a fallback file if the requested file is not found. If the primary requested file is not found and a non-empty fallback file is specified then a warning is logged and the fallback file is looked up. If neither the fallback file can be found then only then an error is logged and the method returns an empty string. This change has also the benefit of putting the common fallback file ("uncalibrated.yaml") pattern to a single place. Signed-off-by: Milan Zamazal <mzamazal@redhat.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com> Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Diffstat (limited to 'src/apps/common/options.h')
0 files changed, 0 insertions, 0 deletions