From d748bdc66d3344761292adc8a611b74e4dfeb88f Mon Sep 17 00:00:00 2001 From: Stefan Klug Date: Wed, 26 Feb 2025 16:59:26 +0100 Subject: ipa: rkisp1: Allow exposure time to be shorter than minimum frame duration limit The minimum FrameDurationLimit also limits the min exposure time and results in overly bright AE regulation. Remove the limit on the minimum exposure time as the vertical blanking ensures the minimum frame duration limit. Fixes: f72c76eb6e06 ("rkisp1: Honor the FrameDurationLimits control") Signed-off-by: Stefan Klug Reviewed-by: Kieran Bingham Tested-by: Kieran Bingham Reviewed-by: Jacopo Mondi --- src/ipa/rkisp1/algorithms/agc.cpp | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/ipa/rkisp1/algorithms/agc.cpp b/src/ipa/rkisp1/algorithms/agc.cpp index 4e0e3734..5a3ba013 100644 --- a/src/ipa/rkisp1/algorithms/agc.cpp +++ b/src/ipa/rkisp1/algorithms/agc.cpp @@ -526,9 +526,7 @@ void Agc::process(IPAContext &context, [[maybe_unused]] const uint32_t frame, double maxAnalogueGain; if (frameContext.agc.autoExposureEnabled) { - minExposureTime = std::clamp(frameContext.agc.minFrameDuration, - context.configuration.sensor.minExposureTime, - context.configuration.sensor.maxExposureTime); + minExposureTime = context.configuration.sensor.minExposureTime; maxExposureTime = std::clamp(frameContext.agc.maxFrameDuration, context.configuration.sensor.minExposureTime, context.configuration.sensor.maxExposureTime); -- cgit v1.2.1