diff --git a/src/fsfw/devicehandlers/AssemblyBase.cpp b/src/fsfw/devicehandlers/AssemblyBase.cpp index ada7c4a8..c22bd4ee 100644 --- a/src/fsfw/devicehandlers/AssemblyBase.cpp +++ b/src/fsfw/devicehandlers/AssemblyBase.cpp @@ -71,7 +71,7 @@ bool AssemblyBase::handleChildrenChangedHealth() { if (iter == childrenMap.end()) { return false; } - HealthState healthState = healthHelper.healthTable->getHealth(iter->first); + HealthState healthState = healthHelper.healthTable->getHealth(convertToDeviceObjectId(iter->first)); if (healthState == HasHealthIF::NEEDS_RECOVERY) { triggerEvent(TRYING_RECOVERY, iter->first, 0); recoveryState = RECOVERY_STARTED; @@ -92,10 +92,15 @@ void AssemblyBase::handleChildrenTransition() { if (commandsOutstanding <= 0) { switch (internalState) { case STATE_NEED_SECOND_STEP: + { internalState = STATE_SECOND_STEP; - commandChildren(targetMode, targetSubmode); + ReturnValue_t result = commandChildren(targetMode, targetSubmode); + if(result == NEED_SECOND_STEP) { + internalState = STATE_NEED_SECOND_STEP; + } return; - case STATE_OVERWRITE_HEALTH: { + } + case STATE_OVERWRITE_HEALTH: { internalState = STATE_SINGLE_STEP; ReturnValue_t result = commandChildren(mode, submode); if (result == NEED_SECOND_STEP) { @@ -170,7 +175,7 @@ ReturnValue_t AssemblyBase::checkChildrenStateOff() { ReturnValue_t AssemblyBase::checkChildOff(uint32_t objectId) { ChildInfo childInfo = childrenMap.find(objectId)->second; - if (healthHelper.healthTable->isCommandable(objectId)) { + if (healthHelper.healthTable->isCommandable(convertToDeviceObjectId(objectId))) { if (childInfo.submode != SUBMODE_NONE) { return returnvalue::FAILED; } else { @@ -227,7 +232,7 @@ bool AssemblyBase::checkAndHandleRecovery() { case RECOVERY_STARTED: // The recovery was already start in #handleChildrenChangedHealth and we just need // to wait for an off time period. - // TODO: make time period configurable + // The timeout can be defined by #setRecoveryWaitTimer recoveryState = RECOVERY_WAIT; recoveryOffTimer.resetTimer(); return true; @@ -264,7 +269,15 @@ void AssemblyBase::overwriteDeviceHealth(object_id_t objectId, HasHealthIF::Heal triggerEvent(OVERWRITING_HEALTH, objectId, oldHealth); internalState = STATE_OVERWRITE_HEALTH; modeHelper.setForced(true); - sendHealthCommand(childrenMap[objectId].commandQueue, EXTERNAL_CONTROL); + if(childrenMap.find(objectId) != childrenMap.end()) { + sendHealthCommand(childrenMap.at(objectId).commandQueue, EXTERNAL_CONTROL); + } else { +#if FSFW_CPP_OSTREAM_ENABLED == 1 + sif::debug << std::hex << SystemObject::getObjectId() << ": invalid mode table entry" + << std::endl; +#endif + } + } void AssemblyBase::triggerModeHelperEvents(Mode_t mode, Submode_t submode) { @@ -274,3 +287,7 @@ void AssemblyBase::triggerModeHelperEvents(Mode_t mode, Submode_t submode) { triggerEvent(CHANGING_MODE, mode, submode); } } + +void AssemblyBase::setRecoveryWaitTimer(uint32_t timeoutMS) { + recoveryOffTimer.setTimeout(timeoutMS); +} diff --git a/src/fsfw/devicehandlers/AssemblyBase.h b/src/fsfw/devicehandlers/AssemblyBase.h index 5e0d826f..ac4feccd 100644 --- a/src/fsfw/devicehandlers/AssemblyBase.h +++ b/src/fsfw/devicehandlers/AssemblyBase.h @@ -206,6 +206,8 @@ class AssemblyBase : public SubsystemBase { void overwriteDeviceHealth(object_id_t objectId, HasHealthIF::HealthState oldHealth); void triggerModeHelperEvents(Mode_t mode, Submode_t submode); + + void setRecoveryWaitTimer(uint32_t timeoutMS); }; #endif /* FSFW_DEVICEHANDLERS_ASSEMBLYBASE_H_ */ diff --git a/src/fsfw/subsystem/SubsystemBase.cpp b/src/fsfw/subsystem/SubsystemBase.cpp index 87fcfebf..7bedfadb 100644 --- a/src/fsfw/subsystem/SubsystemBase.cpp +++ b/src/fsfw/subsystem/SubsystemBase.cpp @@ -78,9 +78,9 @@ void SubsystemBase::executeTable(HybridIterator<ModeListEntry> tableIter, Submod submodeToCommand = targetSubmode; } - if (healthHelper.healthTable->hasHealth(object)) { + if (healthHelper.healthTable->hasHealth(convertToDeviceObjectId(object))) { - switch (healthHelper.healthTable->getHealth(object)) { + switch (healthHelper.healthTable->getHealth(convertToDeviceObjectId(object))) { case NEEDS_RECOVERY: case FAULTY: case PERMANENT_FAULTY: @@ -353,3 +353,7 @@ ReturnValue_t SubsystemBase::registerChild(object_id_t childObjectId, MessageQue } return returnvalue::OK; } + +object_id_t SubsystemBase::convertToDeviceObjectId(object_id_t id) { + return id; +} \ No newline at end of file diff --git a/src/fsfw/subsystem/SubsystemBase.h b/src/fsfw/subsystem/SubsystemBase.h index 072b4ca4..5c21c5fd 100644 --- a/src/fsfw/subsystem/SubsystemBase.h +++ b/src/fsfw/subsystem/SubsystemBase.h @@ -153,6 +153,12 @@ class SubsystemBase : public SystemObject, virtual void announceMode(bool recursive) override; virtual void modeChanged(); + + /** + * @brief Provides an adaptation point for the user to change an objectId into + * a different objectId. + */ + virtual object_id_t convertToDeviceObjectId(object_id_t id); }; #endif /* FSFW_SUBSYSTEM_SUBSYSTEMBASE_H_ */