Index: PersistentAlarm.c =================================================================== diff -u -r05059f677729c710938aca3c6c6df6910b50c24d -r7f049fb55e8f7fd41cf2dbd19a0e43d4e684af78 --- PersistentAlarm.c (.../PersistentAlarm.c) (revision 05059f677729c710938aca3c6c6df6910b50c24d) +++ PersistentAlarm.c (.../PersistentAlarm.c) (revision 7f049fb55e8f7fd41cf2dbd19a0e43d4e684af78) @@ -7,8 +7,8 @@ * * @file PersistentAlarm.c * -* @author (last) Quang Nguyen -* @date (last) 17-Aug-2020 +* @author (last) Sean Nash +* @date (last) 01-Oct-2020 * * @author (original) Quang Nguyen * @date (original) 17-Aug-2020 @@ -17,6 +17,7 @@ #include "AlarmMgmt.h" #include "PersistentAlarm.h" +#include "Timers.h" /** * @addtogroup PersistentAlarm @@ -31,11 +32,11 @@ ALARM_ID_T alarm; ///< Alarm id BOOL isClearable; ///< Flag if alarm can be cleared - U32 persistentClearCount; ///< Persistent count limit before clear alarm - U32 persistentTriggerCount; ///< Persistent count limit before trigger alarm + U32 persistentClearPeriod; ///< Persistent count limit before clear alarm + U32 persistentTriggerPeriod; ///< Persistent count limit before trigger alarm - U32 inRangeCounter; ///< Data in range persistent counter - U32 outOfRangeCounter; ///< Data out of range persistent counter + U32 inRangeStartTime; ///< Data in range persistent counter + U32 outOfRangeStartTime; ///< Data out of range persistent counter } PERSISTENT_ALARM_DATA_T; // ********** private data ********** @@ -53,21 +54,21 @@ * @param alarmIndex Persistent alarm index * @param alarm Alarm id * @param isClearable Flag to indicate alarm is clearable or not - * @param persistentClearCount Persistent count limit before clear alarm - * @param persistentTriggerCount Persistent count limit before trigger alarm + * @param persistentClearPeriod Persistent period limit before clear alarm + * @param persistentTriggerPeriod Persistent period limit before trigger alarm * @return none *************************************************************************/ void initPersistentAlarm( PERSISTENT_ALARM_T alarmIndex, ALARM_ID_T alarm, BOOL isClearable, - F32 persistentClearCount, F32 persistentTriggerCount ) + F32 persistentClearPeriod, F32 persistentTriggerPeriod ) { if ( alarmIndex < NUM_OF_PERSISTENT_ALARM ) { - persistentAlarms[ alarmIndex ].alarm = alarm; - persistentAlarms[ alarmIndex ].isClearable = isClearable; - persistentAlarms[ alarmIndex ].persistentClearCount = persistentClearCount; - persistentAlarms[ alarmIndex ].persistentTriggerCount = persistentTriggerCount; - persistentAlarms[ alarmIndex ].inRangeCounter = 0U; - persistentAlarms[ alarmIndex ].outOfRangeCounter = 0U; + persistentAlarms[ alarmIndex ].alarm = alarm; + persistentAlarms[ alarmIndex ].isClearable = isClearable; + persistentAlarms[ alarmIndex ].persistentClearPeriod = persistentClearPeriod; + persistentAlarms[ alarmIndex ].persistentTriggerPeriod = persistentTriggerPeriod; + persistentAlarms[ alarmIndex ].inRangeStartTime = 0U; + persistentAlarms[ alarmIndex ].outOfRangeStartTime = 0U; } else { @@ -97,18 +98,24 @@ { if ( isOutOfRange ) { - ++persistentAlarms[ alarmIndex ].outOfRangeCounter; - persistentAlarms[ alarmIndex ].inRangeCounter = 0; - if ( persistentAlarms[ alarmIndex ].outOfRangeCounter > persistentAlarms[ alarmIndex ].persistentTriggerCount ) + persistentAlarms[ alarmIndex ].inRangeStartTime = 0; + if ( persistentAlarms[ alarmIndex ].outOfRangeStartTime == 0 ) { + persistentAlarms[ alarmIndex ].outOfRangeStartTime = getMSTimerCount(); + } + if ( didTimeout( persistentAlarms[ alarmIndex ].outOfRangeStartTime, persistentAlarms[ alarmIndex ].persistentTriggerPeriod ) ) + { SET_ALARM_WITH_1_F32_DATA( persistentAlarms[ alarmIndex ].alarm, data ); } } else { - ++persistentAlarms[ alarmIndex ].inRangeCounter; - persistentAlarms[ alarmIndex ].outOfRangeCounter = 0; - BOOL const isInRangePersistent = persistentAlarms[ alarmIndex ].inRangeCounter > persistentAlarms[ alarmIndex ].persistentClearCount; + persistentAlarms[ alarmIndex ].outOfRangeStartTime = 0; + if ( persistentAlarms[ alarmIndex ].inRangeStartTime == 0 ) + { + persistentAlarms[ alarmIndex ].inRangeStartTime = getMSTimerCount(); + } + BOOL const isInRangePersistent = didTimeout( persistentAlarms[ alarmIndex ].inRangeStartTime, persistentAlarms[ alarmIndex ].persistentClearPeriod ); if ( persistentAlarms[ alarmIndex ].isClearable && isInRangePersistent ) { clearAlarm( persistentAlarms[ alarmIndex ].alarm );