Skip to content

Commit 6ab5451

Browse files
committed
Fix #52, Apply consistent Event ID names to common events
1 parent 8653ba6 commit 6ab5451

12 files changed

+60
-59
lines changed

fsw/src/cs_app.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -522,7 +522,7 @@ void CS_ProcessCmd(const CFE_SB_Buffer_t *BufPtr)
522522
break;
523523

524524
default:
525-
CFE_EVS_SendEvent(CS_CC1_ERR_EID, CFE_EVS_EventType_ERROR,
525+
CFE_EVS_SendEvent(CS_CC_ERR_EID, CFE_EVS_EventType_ERROR,
526526
"Invalid ground command code: ID = 0x%08lX, CC = %d",
527527
(unsigned long)CFE_SB_MsgIdToValue(MessageID), CommandCode);
528528

@@ -552,7 +552,7 @@ void CS_HousekeepingCmd(const CS_NoArgsCmd_t *CmdPtr)
552552
CFE_MSG_GetMsgId(&CmdPtr->CmdHeader.Msg, &MessageID);
553553
CFE_MSG_GetFcnCode(&CmdPtr->CmdHeader.Msg, &CommandCode);
554554

555-
CFE_EVS_SendEvent(CS_LEN_ERR_EID, CFE_EVS_EventType_ERROR,
555+
CFE_EVS_SendEvent(CS_CMD_LEN_ERR_EID, CFE_EVS_EventType_ERROR,
556556
"Invalid msg length: ID = 0x%08lX, CC = %d, Len = %lu, Expected = %lu",
557557
(unsigned long)CFE_SB_MsgIdToValue(MessageID), CommandCode, (unsigned long)ActualLength,
558558
(unsigned long)ExpectedLength);

fsw/src/cs_cmds.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,7 @@ void CS_ResetCmd(const CS_NoArgsCmd_t *CmdPtr)
8585
CS_AppData.HkPacket.OSCSErrCounter = 0;
8686
CS_AppData.HkPacket.PassCounter = 0;
8787

88-
CFE_EVS_SendEvent(CS_RESET_DBG_EID, CFE_EVS_EventType_DEBUG, "Reset Counters command recieved");
88+
CFE_EVS_SendEvent(CS_RESET_INF_EID, CFE_EVS_EventType_DEBUG, "Reset Counters command recieved");
8989
}
9090
}
9191

@@ -112,7 +112,7 @@ void CS_BackgroundCheckCycle(const CS_NoArgsCmd_t *CmdPtr)
112112
CFE_MSG_GetMsgId(&CmdPtr->CmdHeader.Msg, &MessageID);
113113
CFE_MSG_GetFcnCode(&CmdPtr->CmdHeader.Msg, &CommandCode);
114114

115-
CFE_EVS_SendEvent(CS_LEN_ERR_EID, CFE_EVS_EventType_ERROR,
115+
CFE_EVS_SendEvent(CS_CMD_LEN_ERR_EID, CFE_EVS_EventType_ERROR,
116116
"Invalid msg length: ID = 0x%08lX, CC = %d, Len = %lu, Expected = %lu",
117117
(unsigned long)CFE_SB_MsgIdToValue(MessageID), CommandCode, (unsigned long)ActualLength,
118118
(unsigned long)ExpectedLength);

fsw/src/cs_events.h

+5-5
Original file line numberDiff line numberDiff line change
@@ -54,13 +54,13 @@
5454
/**
5555
* \brief CS Reset Counters Command Event ID
5656
*
57-
* \par Type: DEBUG
57+
* \par Type: INFORMATION
5858
*
5959
* \par Cause:
6060
*
6161
* This event message is issued when a reset counters command has been received.
6262
*/
63-
#define CS_RESET_DBG_EID 3
63+
#define CS_RESET_INF_EID 3
6464

6565
/**
6666
* \brief CS Disable Checksumming Command Event ID
@@ -451,7 +451,7 @@
451451
* This event message is issued when a software bus message is received
452452
* with an invalid command code.
453453
*/
454-
#define CS_CC1_ERR_EID 34
454+
#define CS_CC_ERR_EID 34
455455

456456
/**
457457
* \brief CS App Termination Event ID
@@ -475,7 +475,7 @@
475475
* This event message is issued when command message is received with a message
476476
* length that doesn't match the expected value.
477477
*/
478-
#define CS_LEN_ERR_EID 36
478+
#define CS_CMD_LEN_ERR_EID 36
479479

480480
/**********************************************************************/
481481
/*EEPROM Commands */
@@ -1466,7 +1466,7 @@
14661466
*
14671467
* This event message is issued when CFE_SB_CreatePipe fails for the command pipe
14681468
*/
1469-
#define CS_INIT_SB_CREATE_ERR_EID 112
1469+
#define CS_CR_PIPE_ERR_EID 112
14701470

14711471
/**
14721472
* \brief CS Software Bus Subscribe To Housekeeping Failed Event ID

fsw/src/cs_init.c

+1-1
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ int32 CS_SbInit(void)
5353
Result = CFE_SB_CreatePipe(&CS_AppData.CmdPipe, CS_AppData.PipeDepth, CS_AppData.PipeName);
5454
if (Result != CFE_SUCCESS)
5555
{
56-
CFE_EVS_SendEvent(CS_INIT_SB_CREATE_ERR_EID, CFE_EVS_EventType_ERROR,
56+
CFE_EVS_SendEvent(CS_CR_PIPE_ERR_EID, CFE_EVS_EventType_ERROR,
5757
"Software Bus Create Pipe for command returned: 0x%08X", (unsigned int)Result);
5858
}
5959
else

0 commit comments

Comments
 (0)