AP_Mount: Do not perform unnecessary processing

ArduPilot · Apr 11, 2024 · a9d0bf8 · a9d0bf8
1 parent 2c267f4
commit a9d0bf8
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/libraries/AP_Mount/AP_Mount_Siyi.cpp b/libraries/AP_Mount/AP_Mount_Siyi.cpp
@@ -38,9 +38,11 @@ void AP_Mount_Siyi::init()
     const AP_SerialManager& serial_manager = AP::serialmanager();
 
     _uart = serial_manager.find_serial(AP_SerialManager::SerialProtocol_Gimbal, 0);
-    if (_uart != nullptr) {
-        _initialised = true;
+    if (_uart == nullptr) {
+        return;
     }
+
+    _initialised = true;
     AP_Mount_Backend::init();
 }
 

diff --git a/libraries/AP_Mount/AP_Mount_Viewpro.cpp b/libraries/AP_Mount/AP_Mount_Viewpro.cpp
@@ -34,10 +34,11 @@ void AP_Mount_Viewpro::init()
     const AP_SerialManager& serial_manager = AP::serialmanager();
 
     _uart = serial_manager.find_serial(AP_SerialManager::SerialProtocol_Gimbal, 0);
-    if (_uart != nullptr) {
-        _initialised = true;
+    if (_uart == nullptr) {
+        return;
     }
 
+    _initialised = true;
     AP_Mount_Backend::init();
 }