@@ -343,6 +343,18 @@ void IPARkISP1::computeParams(const uint32_t frame, const uint32_t bufferId)
{
IPAFrameContext &frameContext = context_.frameContexts.get(frame);
+ /*
+ * \todo: This needs discussion. In raw mode, computeParams is
+ * called without a params buffer, to trigger the setSensorControls
+ * signal. Currently our algorithms don't support prepare calls with
+ * a nullptr. Do we need that or can we safely skip it?
+ */
+ if (bufferId == 0) {
+ ControlList ctrls = getSensorControls(frameContext);
+ setSensorControls.emit(frame, ctrls);
+ return;
+ }
+
RkISP1Params params(context_.configuration.paramFormat,
mappedBuffers_.at(bufferId).planes()[0]);
@@ -1618,6 +1618,10 @@ void PipelineHandlerRkISP1::frameStart(uint32_t sequence)
RkISP1CameraData *data = cameraData(activeCamera_);
data->delayedCtrls_->applyControls(sequence);
+
+ if (isRaw_) {
+ data->ipa_->computeParams(sequence + 1, 0);
+ }
}
bool PipelineHandlerRkISP1::match(DeviceEnumerator *enumerator)
After the pipeline restructuring setSensorControls is no longer emitted within process() but within computeParams(). In raw mode computeParams is not called and therefore setSensorControls is never emitted. Fix that by allowing computeParams to be called with an empty bufferId. This strategy is also used for processStats() to ensure that metadata gets filled in raw mode. Then call computeParams within frameStart() when in raw mode. Signed-off-by: Stefan Klug <stefan.klug@ideasonboard.com> --- src/ipa/rkisp1/rkisp1.cpp | 12 ++++++++++++ src/libcamera/pipeline/rkisp1/rkisp1.cpp | 4 ++++ 2 files changed, 16 insertions(+)