[PATCH] ipa: rpi: common: Avoid warnings when AeEnable control is used
David Plowman
david.plowman at raspberrypi.com
Mon Apr 28 15:16:17 CEST 2025
The AeEnable control is now just a wrapper that is converted to
ExposureTimeMode and AnalogueGainMode controls instead. Therefore, it
should simply be ignored when we encounter it, without the need for
any warnings.
Signed-off-by: David Plowman <david.plowman at raspberrypi.com>
---
src/ipa/rpi/common/ipa_base.cpp | 11 +++++++++++
1 file changed, 11 insertions(+)
diff --git a/src/ipa/rpi/common/ipa_base.cpp b/src/ipa/rpi/common/ipa_base.cpp
index 4c09a093..80c17588 100644
--- a/src/ipa/rpi/common/ipa_base.cpp
+++ b/src/ipa/rpi/common/ipa_base.cpp
@@ -967,6 +967,17 @@ void IpaBase::applyControls(const ControlList &controls)
break;
}
+ case controls::AE_ENABLE: {
+ /*
+ * The AeEnable control is now just a wrapper that will already have been
+ * converted to ExposureTimeMode and AnalogueGainMode equivalents, so there
+ * would be nothing to do here. Nonetheless, "handle" the control so as to
+ * avoid warnings from the "default:" clause of the switch statement.
+ */
+
+ break;
+ }
+
case controls::AE_FLICKER_MODE: {
RPiController::AgcAlgorithm *agc = dynamic_cast<RPiController::AgcAlgorithm *>(
controller_.getAlgorithm("agc"));
--
2.39.5
More information about the libcamera-devel
mailing list