From 4581ce0a8250d5d0cadb32793111c33030f3e127 Mon Sep 17 00:00:00 2001 From: Laurent Pinchart Date: Tue, 15 Mar 2022 15:34:07 +0200 Subject: ipa: ipu3: agc: Use existing local short-hand variable in configure() The configure() function has a local configuration variable referencing context.configuration for the purpose of shortening lines. Use it instead of context.configuration in the remaining locations, and constify it while at it as the configuration isn't meant to be modified. Signed-off-by: Laurent Pinchart Reviewed-by: Jean-Michel Hautbois Reviewed-by: Umang Jain Signed-off-by: Kieran Bingham --- src/ipa/ipu3/algorithms/agc.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'src/ipa/ipu3') diff --git a/src/ipa/ipu3/algorithms/agc.cpp b/src/ipa/ipu3/algorithms/agc.cpp index 70cff3fe..7d4b3503 100644 --- a/src/ipa/ipu3/algorithms/agc.cpp +++ b/src/ipa/ipu3/algorithms/agc.cpp @@ -86,17 +86,17 @@ Agc::Agc() int Agc::configure(IPAContext &context, [[maybe_unused]] const IPAConfigInfo &configInfo) { - IPASessionConfiguration &configuration = context.configuration; + const IPASessionConfiguration &configuration = context.configuration; IPAFrameContext &frameContext = context.frameContext; stride_ = configuration.grid.stride; - minShutterSpeed_ = context.configuration.agc.minShutterSpeed; - maxShutterSpeed_ = std::min(context.configuration.agc.maxShutterSpeed, + minShutterSpeed_ = configuration.agc.minShutterSpeed; + maxShutterSpeed_ = std::min(configuration.agc.maxShutterSpeed, kMaxShutterSpeed); - minAnalogueGain_ = std::max(context.configuration.agc.minAnalogueGain, kMinAnalogueGain); - maxAnalogueGain_ = std::min(context.configuration.agc.maxAnalogueGain, kMaxAnalogueGain); + minAnalogueGain_ = std::max(configuration.agc.minAnalogueGain, kMinAnalogueGain); + maxAnalogueGain_ = std::min(configuration.agc.maxAnalogueGain, kMaxAnalogueGain); /* Configure the default exposure and gain. */ frameContext.agc.gain = std::max(minAnalogueGain_, kMinAnalogueGain); -- cgit v1.2.1