diff --git a/301/CO_Emergency.c b/301/CO_Emergency.c index 8410ba0..d7f5e5a 100644 --- a/301/CO_Emergency.c +++ b/301/CO_Emergency.c @@ -564,32 +564,32 @@ void CO_EM_process(CO_EM_t *em, uint16_t CANerrStChanged = CANerrSt ^ em->CANerrorStatusOld; em->CANerrorStatusOld = CANerrSt; - if (CANerrStChanged & (CO_CAN_ERRTX_WARNING | CO_CAN_ERRRX_WARNING)) { + if ((CANerrStChanged & (CO_CAN_ERRTX_WARNING | CO_CAN_ERRRX_WARNING)) != 0U) { CO_error(em, (CANerrSt & (CO_CAN_ERRTX_WARNING | CO_CAN_ERRRX_WARNING)) != 0U, CO_EM_CAN_BUS_WARNING, CO_EMC_NO_ERROR, 0); } - if (CANerrStChanged & CO_CAN_ERRTX_PASSIVE) { + if ((CANerrStChanged & CO_CAN_ERRTX_PASSIVE) != 0U) { CO_error(em, (CANerrSt & CO_CAN_ERRTX_PASSIVE) != 0U, CO_EM_CAN_TX_BUS_PASSIVE, CO_EMC_CAN_PASSIVE, 0); } - if (CANerrStChanged & CO_CAN_ERRTX_BUS_OFF) { + if ((CANerrStChanged & CO_CAN_ERRTX_BUS_OFF) != 0U) { CO_error(em, (CANerrSt & CO_CAN_ERRTX_BUS_OFF) != 0U, CO_EM_CAN_TX_BUS_OFF, CO_EMC_BUS_OFF_RECOVERED, 0); } - if (CANerrStChanged & CO_CAN_ERRTX_OVERFLOW) { + if ((CANerrStChanged & CO_CAN_ERRTX_OVERFLOW) != 0U) { CO_error(em, (CANerrSt & CO_CAN_ERRTX_OVERFLOW) != 0U, CO_EM_CAN_TX_OVERFLOW, CO_EMC_CAN_OVERRUN, 0); } - if (CANerrStChanged & CO_CAN_ERRTX_PDO_LATE) { + if ((CANerrStChanged & CO_CAN_ERRTX_PDO_LATE) != 0U) { CO_error(em, (CANerrSt & CO_CAN_ERRTX_PDO_LATE) != 0U, CO_EM_TPDO_OUTSIDE_WINDOW, CO_EMC_COMMUNICATION, 0); } - if (CANerrStChanged & CO_CAN_ERRRX_PASSIVE) { + if ((CANerrStChanged & CO_CAN_ERRRX_PASSIVE) != 0U) { CO_error(em, (CANerrSt & CO_CAN_ERRRX_PASSIVE) != 0U, CO_EM_CAN_RX_BUS_PASSIVE, CO_EMC_CAN_PASSIVE, 0); } - if (CANerrStChanged & CO_CAN_ERRRX_OVERFLOW) { + if ((CANerrStChanged & CO_CAN_ERRRX_OVERFLOW) != 0U) { CO_error(em, (CANerrSt & CO_CAN_ERRRX_OVERFLOW) != 0U, CO_EM_CAN_RXB_OVERFLOW, CO_EMC_CAN_OVERRUN, 0); } diff --git a/301/CO_SDOserver.c b/301/CO_SDOserver.c index d734973..bde5289 100644 --- a/301/CO_SDOserver.c +++ b/301/CO_SDOserver.c @@ -809,7 +809,7 @@ CO_SDO_return_t CO_SDOserver_process(CO_SDOserver_t *SDO, if ((SDO->state != CO_SDO_ST_IDLE) && (SDO->state != CO_SDO_ST_ABORT)) { switch (SDO->state) { case CO_SDO_ST_DOWNLOAD_INITIATE_REQ: { - if (SDO->CANrxData[0] & 0x02U) { + if ((SDO->CANrxData[0] & 0x02U) != 0U) { /* Expedited transfer, max 4 bytes of data */ /* Size of OD variable (>0 if indicated) */ @@ -817,7 +817,7 @@ CO_SDO_return_t CO_SDOserver_process(CO_SDOserver_t *SDO, /* Get SDO data size (indicated by SDO client or get from OD) */ OD_size_t dataSizeToWrite = 4; - if (SDO->CANrxData[0] & 0x01U) { + if ((SDO->CANrxData[0] & 0x01U) != 0U) { dataSizeToWrite -= (SDO->CANrxData[0] >> 2) & 0x03U; } else if ((sizeInOd > 0U) && (sizeInOd < 4U)) { @@ -880,7 +880,7 @@ CO_SDO_return_t CO_SDOserver_process(CO_SDOserver_t *SDO, else { #if ((CO_CONFIG_SDO_SRV) & CO_CONFIG_SDO_SRV_SEGMENTED) != 0 /* segmented transfer, is size indicated? */ - if (SDO->CANrxData[0] & 0x01U) { + if ((SDO->CANrxData[0] & 0x01U) != 0U) { uint32_t size; OD_size_t sizeInOd = SDO->OD_IO.stream.dataLength; diff --git a/CANopen.c b/CANopen.c index ed3317d..ced4f9d 100644 --- a/CANopen.c +++ b/CANopen.c @@ -1000,7 +1000,7 @@ CO_ReturnError_t CO_CANopenInit(CO_t *co, #if ((CO_CONFIG_LEDS) & CO_CONFIG_LEDS_ENABLE) != 0 if (CO_GET_CNT(LEDS) == 1U) { err = CO_LEDs_init(co->LEDs); - if (err) { return err; } + if (err != CO_ERROR_NO) { return err; } } #endif @@ -1037,7 +1037,7 @@ CO_ReturnError_t CO_CANopenInit(CO_t *co, #endif nodeId, errInfo); - if (err) { return err; } + if (err != CO_ERROR_NO) { return err; } } /* NMT_Heartbeat */ @@ -1060,7 +1060,7 @@ CO_ReturnError_t CO_CANopenInit(CO_t *co, CO_GET_CO(TX_IDX_HB_PROD), (uint16_t)CO_CAN_ID_HEARTBEAT + nodeId, errInfo); - if (err) { return err; } + if (err != CO_ERROR_NO) { return err; } } #if ((CO_CONFIG_HB_CONS) & CO_CONFIG_HB_CONS_ENABLE) != 0 @@ -1073,7 +1073,7 @@ CO_ReturnError_t CO_CANopenInit(CO_t *co, co->CANmodule, CO_GET_CO(RX_IDX_HB_CONS), errInfo); - if (err) { return err; } + if (err != CO_ERROR_NO) { return err; } } #endif @@ -1088,7 +1088,7 @@ CO_ReturnError_t CO_CANopenInit(CO_t *co, co->CANmodule, CO_GET_CO(TX_IDX_NG_SLV), errInfo); - if (err) { return err; } + if (err != CO_ERROR_NO) { return err; } #endif #if ((CO_CONFIG_NODE_GUARDING) & CO_CONFIG_NODE_GUARDING_MASTER_ENABLE) != 0 err = CO_nodeGuardingMaster_init(co->NGmaster, @@ -1114,7 +1114,7 @@ CO_ReturnError_t CO_CANopenInit(CO_t *co, co->CANmodule, CO_GET_CO(TX_IDX_SDO_SRV) + i, errInfo); - if (err) { return err; } + if (err != CO_ERROR_NO) { return err; } } } @@ -1147,7 +1147,7 @@ CO_ReturnError_t CO_CANopenInit(CO_t *co, CO_GET_CO(TX_IDX_TIME), #endif errInfo); - if (err) { return err; } + if (err != CO_ERROR_NO) { return err; } } #endif @@ -1166,7 +1166,7 @@ CO_ReturnError_t CO_CANopenInit(CO_t *co, CO_GET_CO(TX_IDX_SYNC), #endif errInfo); - if (err) { return err; } + if (err != CO_ERROR_NO) { return err; } } #endif @@ -1278,7 +1278,7 @@ CO_ReturnError_t CO_CANopenInitPDO(CO_t *co, co->CANmodule, CO_GET_CO(RX_IDX_RPDO) + i, errInfo); - if (err) { return err; } + if (err != CO_ERROR_NO) { return err; } } } #endif @@ -1312,7 +1312,7 @@ CO_ReturnError_t CO_CANopenInitPDO(CO_t *co, co->CANmodule, CO_GET_CO(TX_IDX_TPDO) + i, errInfo); - if (err) { return err; } + if (err != CO_ERROR_NO) { return err; } } } #endif @@ -1360,7 +1360,7 @@ CO_ReturnError_t CO_CANopenInitSRDO(CO_t *co, OD_GET(H13FE, OD_H13FE_SRDO_VALID), OD_GET(H13FF, OD_H13FF_SRDO_CHECKSUM), errInfo); - if (err) { return err; } + if (err != CO_ERROR_NO) { return err; } OD_entry_t *SRDOcomm = OD_GET(H1301, OD_H1301_SRDO_1_PARAM); OD_entry_t *SRDOmap = OD_GET(H1381, OD_H1381_SRDO_1_MAPPING); @@ -1388,7 +1388,7 @@ CO_ReturnError_t CO_CANopenInitSRDO(CO_t *co, CANdevTxIdx, CANdevTxIdx + 1U, errInfo); - if (err) { return err; } + if (err != CO_ERROR_NO) { return err; } } CO_SRDO_init_end(co->SRDOGuard);