@@ -21,6 +21,7 @@
#include <libcamera/formats.h>
#include <libcamera/stream.h>
+#include "libcamera/internal/bayer_format.h"
#include "libcamera/internal/framebuffer.h"
#include "libcamera/internal/ipa_manager.h"
#include "libcamera/internal/software_isp/debayer_params.h"
@@ -270,7 +271,16 @@ int SoftwareIsp::configure(const StreamConfiguration &inputCfg,
if (ret < 0)
return ret;
- return debayer_->configure(inputCfg, outputCfgs, ccmEnabled_);
+ ret = debayer_->configure(inputCfg, outputCfgs, ccmEnabled_);
+ if (ret < 0)
+ return ret;
+
+ LOG(SoftwareIsp, Info)
+ << "Input " << inputCfg.size
+ << "-" << BayerFormat::fromPixelFormat(inputCfg.pixelFormat)
+ << " stride " << inputCfg.stride;
+
+ return 0;
}
/**
As shown by commit 94d32fdc55a3 ("pipeline: simple: Consider output sizes when choosing pipe config"), the extra pixel columns CPU debayering requires on the input side makes resolution selection non trivial. Add logging of the selected input config on a successful configure() so that the logs clearly show which sensor mode has been selected. Reviewed-by: Milan Zamazal <mzamazal@redhat.com> Signed-off-by: Hans de Goede <johannes.goede@oss.qualcomm.com> --- Changes in v3: - Move from DebayerCpu::configure() to SoftwareIsp::configure() so that the input fmt also gets logged when using the DebayerEgl class Changes in v2: - New patch in v2 series --- src/libcamera/software_isp/software_isp.cpp | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-)