nikon/CANopenNode
1
0
Fork 0
Code Activity Actions

static analysis: Ignoring return value of functions: CO_setUint8, CO_setUint16, CO_setUint32 [MISRA 2012 Directive 4.7, required]

Browse source
This commit is contained in:
temi54c1l8 2024-06-10 12:07:41 +02:00
parent 187650d073
commit a17b9fa209
5 changed files with 17 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
301/CO_Emergency.c
View file

@ -70,7 +70,7 @@ static ODR_t OD_read_1014(OD_stream_t *stream, void *buf,
CO_CAN_ID_EMERGENCY + em->nodeId : em->producerCanId;
uint32_t COB_IDEmergency32 = em->producerEnabled ? 0 : 0x80000000;
COB_IDEmergency32 |= canId;
CO_setUint32(buf, COB_IDEmergency32);
(void)CO_setUint32(buf, COB_IDEmergency32);
*countRead = sizeof(uint32_t);
@ -139,7 +139,7 @@ static ODR_t OD_read_1014_default(OD_stream_t *stream, void *buf,
uint32_t COB_IDEmergency32 = em->producerEnabled ? 0 : 0x80000000;
COB_IDEmergency32 |= CO_CAN_ID_EMERGENCY + em->nodeId;
CO_setUint32(buf, COB_IDEmergency32);
(void)CO_setUint32(buf, COB_IDEmergency32);
*countRead = sizeof(uint32_t);
return ODR_OK;
@ -194,7 +194,7 @@ static ODR_t OD_read_1003(OD_stream_t *stream, void *buf,
return ODR_DEV_INCOMPAT;
}
if (stream->subIndex == 0) {
CO_setUint8(buf, em->fifoCount);
(void)CO_setUint8(buf, em->fifoCount);
*countRead = sizeof(uint8_t);
return ODR_OK;
@ -210,7 +210,7 @@ static ODR_t OD_read_1003(OD_stream_t *stream, void *buf,
return ODR_DEV_INCOMPAT;
}
else { /* MISRA C 2004 14.10 */ }
CO_setUint32(buf, em->fifo[index].msg);
(void)CO_setUint32(buf, em->fifo[index].msg);
*countRead = sizeof(uint32_t);
return ODR_OK;

6
301/CO_PDO.c
View file

@ -430,7 +430,7 @@ static ODR_t OD_read_PDO_commParam(OD_stream_t *stream, void *buf,
/* If PDO is not valid, set bit 31 */
if (!PDO->valid) { COB_ID |= 0x80000000; }
CO_setUint32(buf, COB_ID);
(void)CO_setUint32(buf, COB_ID);
}
return returnCode;
@ -552,7 +552,7 @@ static ODR_t OD_write_14xx(OD_stream_t *stream, const void *buf,
if (valid != PDO->valid || CAN_ID != PDO->configuredCanId) {
/* if default CAN-ID is written, store to OD without Node-ID */
if (CAN_ID == PDO->preDefinedCanId) {
CO_setUint32(bufCopy, COB_ID & 0xFFFFFF80);
(void)CO_setUint32(bufCopy, COB_ID & 0xFFFFFF80);
}
if (!valid) {
CAN_ID = 0;
@ -983,7 +983,7 @@ static ODR_t OD_write_18xx(OD_stream_t *stream, const void *buf,
if (valid != PDO->valid || CAN_ID != PDO->configuredCanId) {
/* if default CAN-ID is written, store to OD without Node-ID */
if (CAN_ID == PDO->preDefinedCanId) {
CO_setUint32(bufCopy, COB_ID & 0xFFFFFF80);
(void)CO_setUint32(bufCopy, COB_ID & 0xFFFFFF80);
}
if (!valid) {
CAN_ID = 0;

4
301/CO_TIME.c
View file

@ -223,8 +223,8 @@ bool_t CO_TIME_process(CO_TIME_t *TIME,
uint32_t ms_swapped = CO_SWAP_32(TIME->ms);
uint16_t days_swapped = CO_SWAP_16(TIME->days);
CO_setUint32(&TIME->CANtxBuff->data[0], ms_swapped);
CO_setUint16(&TIME->CANtxBuff->data[4], days_swapped);
(void)CO_setUint32(&TIME->CANtxBuff->data[0], ms_swapped);
(void)CO_setUint16(&TIME->CANtxBuff->data[4], days_swapped);
CO_CANsend(TIME->CANdevTx, TIME->CANtxBuff);
}
else {

4
304/CO_SRDO.c
View file

@ -177,7 +177,7 @@ OD_read_SRDO_communicationParam(OD_stream_t* stream, void* buf, OD_size_t count,
value += (uint32_t)SRDO->nodeId * 2;
}
CO_setUint32(buf, value);
(void)CO_setUint32(buf, value);
}
return returnCode;
@ -240,7 +240,7 @@ OD_write_SRDO_communicationParam(OD_stream_t* stream, const void* buf, OD_size_t
/* if default COB-ID is being written, write defaultCOB_ID without nodeId */
if ((SRDO->nodeId <= 64U) && (value == (defaultCOB_ID + ((uint32_t)SRDO->nodeId * 2)))) {
value = defaultCOB_ID;
CO_setUint32(bufCopy, value);
(void)CO_setUint32(bufCopy, value);
}
} else { /* MISRA C 2004 14.10 */
}

12
305/CO_LSSmaster.c
View file

@ -224,16 +224,16 @@ static CO_LSSmaster_return_t CO_LSSmaster_switchStateSelectInitiate(
CO_FLAG_CLEAR(LSSmaster->CANrxNew);
(void)memset(&LSSmaster->TXbuff->data[6], 0, sizeof(LSSmaster->TXbuff->data) - 6);
LSSmaster->TXbuff->data[0] = CO_LSS_SWITCH_STATE_SEL_VENDOR;
CO_setUint32(&LSSmaster->TXbuff->data[1], lssAddress->identity.vendorID);
(void)CO_setUint32(&LSSmaster->TXbuff->data[1], lssAddress->identity.vendorID);
CO_CANsend(LSSmaster->CANdevTx, LSSmaster->TXbuff);
LSSmaster->TXbuff->data[0] = CO_LSS_SWITCH_STATE_SEL_PRODUCT;
CO_setUint32(&LSSmaster->TXbuff->data[1], lssAddress->identity.productCode);
(void)CO_setUint32(&LSSmaster->TXbuff->data[1], lssAddress->identity.productCode);
CO_CANsend(LSSmaster->CANdevTx, LSSmaster->TXbuff);
LSSmaster->TXbuff->data[0] = CO_LSS_SWITCH_STATE_SEL_REV;
CO_setUint32(&LSSmaster->TXbuff->data[1], lssAddress->identity.revisionNumber);
(void)CO_setUint32(&LSSmaster->TXbuff->data[1], lssAddress->identity.revisionNumber);
CO_CANsend(LSSmaster->CANdevTx, LSSmaster->TXbuff);
LSSmaster->TXbuff->data[0] = CO_LSS_SWITCH_STATE_SEL_SERIAL;
CO_setUint32(&LSSmaster->TXbuff->data[1], lssAddress->identity.serialNumber);
(void)CO_setUint32(&LSSmaster->TXbuff->data[1], lssAddress->identity.serialNumber);
CO_CANsend(LSSmaster->CANdevTx, LSSmaster->TXbuff);
ret = CO_LSSmaster_WAIT_SLAVE;
@ -565,7 +565,7 @@ CO_LSSmaster_return_t CO_LSSmaster_ActivateBit(
CO_FLAG_CLEAR(LSSmaster->CANrxNew);
LSSmaster->TXbuff->data[0] = CO_LSS_CFG_ACTIVATE_BIT_TIMING;
CO_setUint16(&LSSmaster->TXbuff->data[1], switchDelay_ms);
(void)CO_setUint16(&LSSmaster->TXbuff->data[1], switchDelay_ms);
(void)memset(&LSSmaster->TXbuff->data[3], 0, sizeof(LSSmaster->TXbuff->data) - 3);
CO_CANsend(LSSmaster->CANdevTx, LSSmaster->TXbuff);
@ -757,7 +757,7 @@ static void CO_LSSmaster_FsSendMsg(
CO_FLAG_CLEAR(LSSmaster->CANrxNew);
LSSmaster->TXbuff->data[0] = CO_LSS_IDENT_FASTSCAN;
CO_setUint32(&LSSmaster->TXbuff->data[1], idNumber);
(void)CO_setUint32(&LSSmaster->TXbuff->data[1], idNumber);
LSSmaster->TXbuff->data[5] = bitCheck;
LSSmaster->TXbuff->data[6] = lssSub;
LSSmaster->TXbuff->data[7] = lssNext;