summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Plowman <david.plowman@raspberrypi.com>2025-04-28 14:16:17 +0100
committerKieran Bingham <kieran.bingham@ideasonboard.com>2025-04-29 16:14:49 +0100
commite4677362a162b6b2174da3c844353f3321068ed0 (patch)
tree901de03ddb42b2c5b7cedac4643eb0e67e24b43c
parent17e41b2a3a144a80e60f28e15a4c2aaa42c87105 (diff)
ipa: rpi: common: Avoid warnings when AeEnable control is usedHEADmaster
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@raspberrypi.com> Reviewed-by: Naushir Patuck <naush@raspberrypi.com> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com> Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
-rw-r--r--src/ipa/rpi/common/ipa_base.cpp11
1 files changed, 11 insertions, 0 deletions
diff --git a/src/ipa/rpi/common/ipa_base.cpp b/src/ipa/rpi/common/ipa_base.cpp
index 6734c32e..e0a93daa 100644
--- a/src/ipa/rpi/common/ipa_base.cpp
+++ b/src/ipa/rpi/common/ipa_base.cpp
@@ -946,6 +946,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"));