software_isp: Log input config from configure()

As shown by commit 94d32fdc55 ("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>
Reviewed-by: Barnabás Pőcze <barnabas.pocze@ideasonboard.com>
Signed-off-by: Hans de Goede <johannes.goede@oss.qualcomm.com>
Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
This commit is contained in:
Hans de Goede
2026-03-10 13:01:05 +01:00
committed by Kieran Bingham
parent e948ada380
commit c2f0aeb5fc

View File

@@ -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;
}
/**