Com Subsystem #364

Merged
muellerr merged 26 commits from meier/com-ss into develop 2023-02-23 15:22:47 +01:00
3 changed files with 4 additions and 2 deletions
Showing only changes of commit bf65d13849 - Show all commits

View File

@ -164,6 +164,7 @@ ReturnValue_t PdecHandler::irqOperation() {
state = State::WAIT_FOR_RECOVERY;
return result;
}
checkLocks();
state = State::RUNNING;
break;
case State::RUNNING: {
Review

Any specific reason or bug why this was added?

Any specific reason or bug why this was added?
Review

Yes. If the syrlinks already had a bit lock during reboot, the bit lock and carrier lock events were never triggered.

Yes. If the syrlinks already had a bit lock during reboot, the bit lock and carrier lock events were never triggered.

View File

@ -67,7 +67,7 @@ Subsystem& satsystem::com::init() {
buildTxAndRxLowRateSequence(SUBSYSTEM, entry);
buildTxAndRxHighRateSequence(SUBSYSTEM, entry);
buildTxAndRxDefaultRateSequence(SUBSYSTEM, entry);
SUBSYSTEM.setInitialMode(NML, ::com::Submode::RX_ONLY);
SUBSYSTEM.setInitialMode(COM_SEQUENCE_RX_ONLY.first);
return SUBSYSTEM;
}

View File

@ -3,7 +3,7 @@
#include <fsfw/devicehandlers/DeviceHandlerIF.h>
#include <fsfw/subsystem/Subsystem.h>
#include <mission/acsDefs.h>
#include "mission/comDefs.h"
#include "acsModeTree.h"
#include "comModeTree.h"
#include "eive/objects.h"
@ -85,6 +85,7 @@ void buildSafeSequence(Subsystem& ss, ModeListEntry& eh) {
// Build SAFE transition 0. Two transitions to reduce number of consecutive events and because
// consecutive commanding of TCS and ACS can lead to SPI issues.
iht(objects::TCS_SUBSYSTEM, NML, 0, EIVE_TABLE_SAFE_TRANS_0.second);
iht(objects::COM_SUBSYSTEM, com::RX_ONLY, 0, EIVE_TABLE_SAFE_TRANS_0.second);
check(ss.addTable(TableEntry(EIVE_TABLE_SAFE_TRANS_0.first, &EIVE_TABLE_SAFE_TRANS_0.second)),
ctxc);