[v5,23/47] ipa: libipa: agc: Omit `setLimits()` in `configure()`
diff mbox series

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

Commit Message

Barnabás Pőcze Aug. 17, 2026, 11:43 a.m. UTC 
`AgcMeanLuminance::setLimits()` is only required to be called before
`calculateNewEv()`, so the call in `configure()` has no real effect
because no calculation is carried out. So remove it.

Signed-off-by: Barnabás Pőcze <barnabas.pocze@ideasonboard.com>
Reviewed-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>
Reviewed-by: Stefan Klug <stefan.klug@ideasonboard.com>
---
 src/ipa/libipa/agc.cpp | 3 ---
 1 file changed, 3 deletions(-)

Patch
diff mbox series 

diff --git a/src/ipa/libipa/agc.cpp b/src/ipa/libipa/agc.cpp
index 1d12ec6b1f..8c8ea41ea9 100644
--- a/src/ipa/libipa/agc.cpp
+++ b/src/ipa/libipa/agc.cpp
@@ -346,9 +346,6 @@  int AgcAlgorithm::configure(agc::Session &session, agc::ActiveState &state,
 	session.maxFrameDuration = std::chrono::microseconds(frameDurations[1]);
 
 	impl_.configure(session.lineDuration, config.sensor);
-	impl_.setLimits(session.minExposureTime, session.maxExposureTime,
-			session.minAnalogueGain, session.maxAnalogueGain,
-			{});
 	impl_.resetFrameCount();
 
 	/* Configure the default exposure and gain. */