[ARP] Fixed SMController start

src/boards/Groundstation/Automated/SMController/SMController.cpp

@@ -55,16 +55,10 @@ SMController::SMController(TaskScheduler* sched)
 
 bool SMController::start()
 {
-    size_t result;
-    bool ok = true;
-
-    // add the update task
-    result = scheduler->addTask(bind(&SMController::update, this),
+    size_t result = scheduler->addTask(bind(&SMController::update, this),
                                        SMControllerConfig::UPDATE_PERIOD,
                                        TaskScheduler::Policy::RECOVER);
-    ok &= result != 0;
-
-    return ActiveObject::start() && ok;
+    return HSM::start() && result != 0;
 }
 
 void SMController::setAntennaCoordinates(

src/boards/Groundstation/Automated/SMController/SMControllerData.h

@@ -60,13 +60,14 @@ enum class SMControllerState : uint8_t
      * @brief macro state for no feedback (armed_nf, test_nf,
      * fix_rocket_nf, active_nf)
      */
-    NO_FEEDBACK
+    NO_FEEDBACK,
+    INVALID,
 };
 
 struct SMControllerStatus
 {
-    uint64_t timestamp;
-    SMControllerState state;
+    uint64_t timestamp      = 0;
+    SMControllerState state = SMControllerState::INVALID;
 
     static std::string header() { return "timestamp,state\n"; }