changed some error messages to warnings
This commit is contained in:
parent
53a780ed0f
commit
4ccec03e47
@ -448,7 +448,7 @@ ReturnValue_t PlocMPSoCHandler::handleAckReport(const uint8_t* data) {
|
|||||||
|
|
||||||
result = verifyPacket(data, mpsoc::SIZE_ACK_REPORT);
|
result = verifyPacket(data, mpsoc::SIZE_ACK_REPORT);
|
||||||
if (result == MPSoCReturnValuesIF::CRC_FAILURE) {
|
if (result == MPSoCReturnValuesIF::CRC_FAILURE) {
|
||||||
sif::error << "PlocMPSoCHandler::handleAckReport: CRC failure" << std::endl;
|
sif::warning << "PlocMPSoCHandler::handleAckReport: CRC failure" << std::endl;
|
||||||
nextReplyId = mpsoc::NONE;
|
nextReplyId = mpsoc::NONE;
|
||||||
replyRawReplyIfnotWiretapped(data, mpsoc::SIZE_ACK_REPORT);
|
replyRawReplyIfnotWiretapped(data, mpsoc::SIZE_ACK_REPORT);
|
||||||
triggerEvent(MPSOC_HANDLER_CRC_FAILURE);
|
triggerEvent(MPSOC_HANDLER_CRC_FAILURE);
|
||||||
@ -493,7 +493,7 @@ ReturnValue_t PlocMPSoCHandler::handleExecutionReport(const uint8_t* data) {
|
|||||||
|
|
||||||
result = verifyPacket(data, mpsoc::SIZE_EXE_REPORT);
|
result = verifyPacket(data, mpsoc::SIZE_EXE_REPORT);
|
||||||
if (result == MPSoCReturnValuesIF::CRC_FAILURE) {
|
if (result == MPSoCReturnValuesIF::CRC_FAILURE) {
|
||||||
sif::error << "PlocMPSoCHandler::handleExecutionReport: CRC failure" << std::endl;
|
sif::warning << "PlocMPSoCHandler::handleExecutionReport: CRC failure" << std::endl;
|
||||||
nextReplyId = mpsoc::NONE;
|
nextReplyId = mpsoc::NONE;
|
||||||
return result;
|
return result;
|
||||||
}
|
}
|
||||||
@ -506,11 +506,12 @@ ReturnValue_t PlocMPSoCHandler::handleExecutionReport(const uint8_t* data) {
|
|||||||
}
|
}
|
||||||
case (mpsoc::apid::EXE_FAILURE): {
|
case (mpsoc::apid::EXE_FAILURE): {
|
||||||
// TODO: Interpretation of status field in execution report
|
// TODO: Interpretation of status field in execution report
|
||||||
sif::error << "PlocMPSoCHandler::handleExecutionReport: Received execution failure report"
|
sif::warning << "PlocMPSoCHandler::handleExecutionReport: Received execution failure report"
|
||||||
<< std::endl;
|
<< std::endl;
|
||||||
DeviceCommandId_t commandId = getPendingCommand();
|
DeviceCommandId_t commandId = getPendingCommand();
|
||||||
if (commandId != DeviceHandlerIF::NO_COMMAND_ID) {
|
if (commandId != DeviceHandlerIF::NO_COMMAND_ID) {
|
||||||
triggerEvent(EXE_FAILURE, commandId);
|
uint16_t status = getStatus(data);
|
||||||
|
triggerEvent(EXE_FAILURE, commandId, status);
|
||||||
} else {
|
} else {
|
||||||
sif::debug << "PlocMPSoCHandler::handleExecutionReport: Unknown command id" << std::endl;
|
sif::debug << "PlocMPSoCHandler::handleExecutionReport: Unknown command id" << std::endl;
|
||||||
}
|
}
|
||||||
@ -521,7 +522,7 @@ ReturnValue_t PlocMPSoCHandler::handleExecutionReport(const uint8_t* data) {
|
|||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
default: {
|
default: {
|
||||||
sif::error << "PlocMPSoCHandler::handleExecutionReport: Unknown APID" << std::endl;
|
sif::warning << "PlocMPSoCHandler::handleExecutionReport: Unknown APID" << std::endl;
|
||||||
result = RETURN_FAILED;
|
result = RETURN_FAILED;
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
@ -534,7 +535,7 @@ ReturnValue_t PlocMPSoCHandler::handleMemoryReadReport(const uint8_t* data) {
|
|||||||
ReturnValue_t result = RETURN_OK;
|
ReturnValue_t result = RETURN_OK;
|
||||||
result = verifyPacket(data, tmMemReadReport.rememberRequestedSize);
|
result = verifyPacket(data, tmMemReadReport.rememberRequestedSize);
|
||||||
if (result == MPSoCReturnValuesIF::CRC_FAILURE) {
|
if (result == MPSoCReturnValuesIF::CRC_FAILURE) {
|
||||||
sif::error << "PlocMPSoCHandler::handleMemoryReadReport: Memory read report has invalid crc"
|
sif::warning << "PlocMPSoCHandler::handleMemoryReadReport: Memory read report has invalid crc"
|
||||||
<< std::endl;
|
<< std::endl;
|
||||||
}
|
}
|
||||||
uint16_t memLen =
|
uint16_t memLen =
|
||||||
@ -751,6 +752,6 @@ void PlocMPSoCHandler::printStatus(const uint8_t* data) {
|
|||||||
sif::info << "Verification report status: 0x" << std::hex << status << std::endl;
|
sif::info << "Verification report status: 0x" << std::hex << status << std::endl;
|
||||||
}
|
}
|
||||||
|
|
||||||
void PlocMPSoCHandler::getStatus(const uint8_t* data) {
|
uint16_t PlocMPSoCHandler::getStatus(const uint8_t* data) {
|
||||||
return *(data + STATUS_OFFSET) << 8 | *(data + STATUS_OFFSET + 1);
|
return *(data + STATUS_OFFSET) << 8 | *(data + STATUS_OFFSET + 1);
|
||||||
}
|
}
|
||||||
|
@ -62,8 +62,12 @@ private:
|
|||||||
//! [EXPORT] : [COMMENT] PLOC crc failure in telemetry packet
|
//! [EXPORT] : [COMMENT] PLOC crc failure in telemetry packet
|
||||||
static const Event MEMORY_READ_RPT_CRC_FAILURE = MAKE_EVENT(1, severity::LOW);
|
static const Event MEMORY_READ_RPT_CRC_FAILURE = MAKE_EVENT(1, severity::LOW);
|
||||||
//! [EXPORT] : [COMMENT] PLOC receive acknowledgment failure report
|
//! [EXPORT] : [COMMENT] PLOC receive acknowledgment failure report
|
||||||
|
//! P1: Command Id which leads the acknowledgment failure report
|
||||||
|
//! P2: The status field inserted by the MPSoC into the data field
|
||||||
static const Event ACK_FAILURE = MAKE_EVENT(2, severity::LOW);
|
static const Event ACK_FAILURE = MAKE_EVENT(2, severity::LOW);
|
||||||
//! [EXPORT] : [COMMENT] PLOC receive execution failure report
|
//! [EXPORT] : [COMMENT] PLOC receive execution failure report
|
||||||
|
//! P1: Command Id which leads the execution failure report
|
||||||
|
//! P2: The status field inserted by the MPSoC into the data field
|
||||||
static const Event EXE_FAILURE = MAKE_EVENT(3, severity::LOW);
|
static const Event EXE_FAILURE = MAKE_EVENT(3, severity::LOW);
|
||||||
//! [EXPORT] : [COMMENT] PLOC reply has invalid crc
|
//! [EXPORT] : [COMMENT] PLOC reply has invalid crc
|
||||||
static const Event MPSOC_HANDLER_CRC_FAILURE = MAKE_EVENT(4, severity::LOW);
|
static const Event MPSOC_HANDLER_CRC_FAILURE = MAKE_EVENT(4, severity::LOW);
|
||||||
|
2
tmtc
2
tmtc
@ -1 +1 @@
|
|||||||
Subproject commit 63b35da15c5f0a20c6f689c7fb029c2712e0935e
|
Subproject commit 6572a5df6169772b517f2de2f3e4a1ba8f6471f4
|
Loading…
Reference in New Issue
Block a user