[v7,26/47] ipa: libipa: agc: Adjust debug messages slightly
diff mbox series

Message ID 20260821130218.2706196-27-barnabas.pocze@ideasonboard.com
State New
Headers show
Series
  • ipa: libipa: agc rework
Related show

Commit Message

Barnabás Pőcze Aug. 21, 2026, 1:01 p.m. UTC 
The log message showing the result of AGC seems too verbose to me,
and the labels and values are far from each other. And also log the
line duration when showing exposure and gain limits.

Signed-off-by: Barnabás Pőcze <barnabas.pocze@ideasonboard.com>
Reviewed-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>
---
 src/ipa/libipa/agc.cpp | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

Patch
diff mbox series 

diff --git a/src/ipa/libipa/agc.cpp b/src/ipa/libipa/agc.cpp
index 3f54dc99f2..e593eac64a 100644
--- a/src/ipa/libipa/agc.cpp
+++ b/src/ipa/libipa/agc.cpp
@@ -321,8 +321,10 @@  int AgcAlgorithm::configure(agc::Session &session, agc::ActiveState &state,
 	float defGain = config.sensor->gain(v4l2Gain.def().get<int32_t>());
 
 	LOG(Agc, Debug)
-		<< "Exposure: [" << minExposure << ", " << maxExposure
-		<< "], gain: [" << minGain << ", " << maxGain << "]";
+		<< "exposure: [" << minExposure << ',' << maxExposure << "], "
+		<< "gain: [" << minGain << ',' << maxGain << "], "
+		<< "line-duration: " << session.lineDuration << ", "
+		<< "sensor-output: " << session.sensor.outputSize;
 
 	/*
 	 * Compute the frame duration limits.
@@ -670,10 +672,10 @@  void AgcAlgorithm::process(const agc::Session &session, agc::ActiveState &state,
 	state.automatic.yTarget = newEv.yTarget;
 
 	LOG(Agc, Debug)
-		<< "Divided up exposure time, analogue gain, quantization gain"
-		<< " and digital gain are " << newEv.exposureTime
-		<< ", " << state.automatic.gain << ", " << state.automatic.quantizationGain
-		<< " and " << newEv.digitalGain;
+		<< "exposure-time: " << newEv.exposureTime << ", "
+		<< "analogue-gain: " << state.automatic.gain << ", "
+		<< "quantization-gain: " << state.automatic.quantizationGain << ", "
+		<< "digital-gain: " << newEv.digitalGain;
 
 	/*
 	 * Expand the target frame duration so that we do not run faster than