Merge branch 'develop' into meier/plocSupvUpdate

bsp_q7s/core/CoreController.cpp

@@ -8,7 +8,7 @@
 #include "fsfw/serviceinterface/ServiceInterface.h"
 #include "fsfw/timemanager/Stopwatch.h"
 #include "fsfw/version.h"
-#include "watchdogConf.h"
+#include "watchdog/definitions.h"
 #if OBSW_USE_TMTC_TCP_BRIDGE == 0
 #include "fsfw/osal/common/UdpTmTcBridge.h"
 #else

bsp_q7s/core/obsw.cpp

@@ -8,7 +8,7 @@
 #include "OBSWVersion.h"
 #include "fsfw/tasks/TaskFactory.h"
 #include "fsfw/version.h"
-#include "watchdogConf.h"
+#include "watchdog/definitions.h"
 
 static int OBSW_ALREADY_RUNNING = -2;
 

mission/devices/IMTQHandler.cpp

@@ -10,7 +10,6 @@
 IMTQHandler::IMTQHandler(object_id_t objectId, object_id_t comIF, CookieIF* comCookie,
                          power::Switch_t pwrSwitcher)
     : DeviceHandlerBase(objectId, comIF, comCookie),
-      switcher(pwrSwitcher),
       engHkDataset(this),
       calMtmMeasurementSet(this),
       rawMtmMeasurementSet(this),
@@ -19,7 +18,8 @@ IMTQHandler::IMTQHandler(object_id_t objectId, object_id_t comIF, CookieIF* comC
       posYselfTestDataset(this),
       negYselfTestDataset(this),
       posZselfTestDataset(this),
-      negZselfTestDataset(this) {
+      negZselfTestDataset(this),
+      switcher(pwrSwitcher) {
   if (comCookie == NULL) {
     sif::error << "IMTQHandler: Invalid com cookie" << std::endl;
   }

mission/devices/PCDUHandler.cpp

@@ -81,12 +81,16 @@ ReturnValue_t PCDUHandler::initialize() {
 
 void PCDUHandler::initializeSwitchStates() {
   using namespace pcdu;
-  for (uint8_t idx = 0; idx < NUMBER_OF_SWITCHES; idx++) {
-    if (idx < PDU::CHANNELS_LEN) {
-      switchStates[idx] = INIT_SWITCHES_PDU1[idx];
-    } else {
-      switchStates[idx] = INIT_SWITCHES_PDU2[idx];
+  try {
+    for (uint8_t idx = 0; idx < NUMBER_OF_SWITCHES; idx++) {
+      if (idx < PDU::CHANNELS_LEN) {
+        switchStates[idx] = INIT_SWITCHES_PDU1.at(idx);
+      } else {
+        switchStates[idx] = INIT_SWITCHES_PDU2.at(idx - PDU::CHANNELS_LEN);
+      }
     }
+  } catch (const std::out_of_range& err) {
+    sif::error << "PCDUHandler::initializeSwitchStates: " << err.what() << std::endl;
   }
 }
 

scripts/q7s-cp.py

@@ -92,7 +92,7 @@ def build_cmd(args):
         else:
             target = args.target
     if args.invert:
-        cmd += f"{port_args} {address}:{args.source} {target}"
+        cmd += f"{port_args} {address}:{source_files} {target}"
     else:
         cmd += f"{port_args} {source_files} {address}:{target}"
     return cmd

watchdog/Watchdog.cpp

@@ -1,5 +1,5 @@
 #include "Watchdog.h"
-#include "watchdogConf.h"
+#include "definitions.h"
 
 #include <errno.h>
 #include <sys/types.h>

watchdog/definitions.h

@@ -1,6 +1,8 @@
 #ifndef WATCHDOG_DEFINITIONS_H_
 #define WATCHDOG_DEFINITIONS_H_
 
+#include <watchdogConf.h>
+
 namespace watchdog {
 
 // Suspend watchdog operations temporarily

watchdog/watchdogConf.h.in

@@ -1,8 +1,6 @@
 #include <cstdint>
 #include <string>
 
-#include "watchdog/definitions.h"
-
 #define WATCHDOG_VERBOSE_LEVEL              1
 /**
  * This flag instructs the watchdog to create a special file in /tmp if the OBSW is running