1
0
Fork 0
Commit graph

75 commits

Author SHA1 Message Date
Janez
8c09a433ef Fix CANopen terminology: “CAN messages” are actually “CAN frames”
Fix all documentation and comments according to CiA proposals:
https://www.can-cia.org/services/publications/can-community-news/05-2025

CANopenNode was using "CAN message" in many places, but actually such
term is incorrect. No code or variable names was changed.
2026-06-26 10:23:01 +02:00
Janez
05931a6b76 Fix and update doxygen documentation. 2024-08-07 00:17:39 +02:00
Janez
82c95a2bc9 Format comments in the .c files. 2024-07-06 21:48:21 +02:00
Janez
95c1705e76 Format the .c files using clang-format v15. 2024-07-06 19:57:06 +02:00
Janez
0d016a0019 Remove /*********...*/ comment delimiters. 2024-07-06 18:53:40 +02:00
Janez
f064eeaa40 Format the license information in files. 2024-07-06 13:03:00 +02:00
Janez
fcd89f87de Remove comments from arguments in CO_CANrxBufferInit and CO_CANtxBufferInit 2024-07-05 18:38:01 +02:00
temi54c1l8
baa9e4e2b2 CO_Emergency: static analysis: argument 1 of type 'void *' is not compatible with argument 2 of type 'uint8_t *' (aka 'unsigned char *') in call to function 'memcpy' 2024-06-27 12:27:01 +02:00
temi54c1l8
a501f961d4 CO_Emergency: ignoring return value of functions [MISRA 2012 Directive 4.7, required], [MISRA 2012 Rule 17.7, required] 2024-06-27 12:24:20 +02:00
temi54c1l8
d9d9044221 CO_Emergency: static analysis: right operand to == is a composite expression of type 'unsigned8' which is smaller than the left operand of type 'unsigned16' [MISRA 2012 Rule 10.7, required] 2024-06-27 12:22:43 +02:00
temi54c1l8
58606174a2 CO_Emergency: static analysis: side effects on right hand of logical operator, '&&' [MISRA 2012 Rule 13.5, required] 2024-06-27 12:21:24 +02:00
temi54c1l8
42c39a2b51 CO_Emergency: static analysis: an unsigned value and a signed value cannot be used together as operands [MISRA 2012 Rule 10.4, required] 2024-06-27 12:19:07 +02:00
temi54c1l8
ce405b1e27 CO_Emergency: static analysis: body should be a compound statement [MISRA 2012 Rule 15.6, required] 2024-06-27 12:12:57 +02:00
temi54c1l8
7bf5065e41 CO_Emergency: static analysis: no 'else' at end of 'if ... else if' chain [MISRA 2012 Rule 15.7, required] 2024-06-27 12:11:19 +02:00
temi54c1l8
ca968aa160 CO_Emergency: static analysis: conditional of #if does not evaluate to 0 or 1 [MISRA 2012 Rule 20.8, required] 2024-06-27 12:08:42 +02:00
temi54c1l8
d76aeb178f CO_Emergency: static analysis: unsigned integer literal without a 'U' suffix [MISRA 2012 Rule 7.2, required] 2024-06-27 12:07:42 +02:00
temi54c1l8
071f6d7935 CO_Emergency: static analysis: dependence placed on operator precedence [MISRA 2012 Rule 12.1, advisory] 2024-06-27 12:06:07 +02:00
temi54c1l8
f9d8bdeff3 static analysys: local variable 'data' could be pointer to const [MISRA 2012 Rule 8.13, advisory]
refactory CO_CANrxMsg_readData return const
2024-06-14 11:27:22 +02:00
temi54c1l8
41c9e742f8 CO_Emergency: static analysis: named parameter 'timeDifference_us' of 'non-virtual' function 'CO_EM_process' not subsequently referenced [MISRA 2012 Rule 2.7, advisory] 2024-06-14 10:59:43 +02:00
temi54c1l8
ab26145597 CO_Emergency: static analysis: side effects on right hand of logical operator, '&&' [MISRA 2012 Rule 13.5, required]
because em->CANtxBuff->bufferFull is volatile
2024-06-14 08:56:03 +02:00
temi54c1l8
f48d5058e6 CO_Emergency: static analysis: increment/decrement operation combined with other operation with side-effects [MISRA 2012 Rule 13.3, advisory] 2024-06-13 11:05:14 +02:00
temi54c1l8
9bffb7d789 CO_Emergency: static analysis: argument 1 of type 'uint8_t [8]' is not compatible with argument 2 of type 'uint32_t *' (aka 'unsigned int *') in call to function 'memcpy' 2024-06-13 11:02:22 +02:00
temi54c1l8
be2a4690d6 CO_Emergency: static analysis: a signed value is not an appropriate [MISRA 2012 Rule 10.1, required] 2024-06-13 10:56:17 +02:00
temi54c1l8
266f77df42 CO_Emergency: static analysis: cannot assign 'unsigned32' to narrower essential type 'unsigned16' [MISRA 2012 Rule 10.3, required] 2024-06-13 10:45:25 +02:00
temi54c1l8
cff7a6f962 CO_Emergency: static analysis: cannot assign a composite expression of type 'unsigned8' to an object of wider type 'unsigned32' [MISRA 2012 Rule 10.6, required] 2024-06-13 10:43:49 +02:00
temi54c1l8
686ab04ee9 static analysis: enumerator 'CO_CAN_ID_EMERGENCY' reuses the constant value '128' previously used by enumerator 'CO_CAN_ID_SYNC'
refactoring changed enum with define
2024-06-12 14:26:51 +02:00
temi54c1l8
691728f71a static analysis: essential type of condition of 'if' statement is in not correct [MISRA 2012 Rule 14.4, required] 2024-06-12 13:57:42 +02:00
temi54c1l8
7855d35bae static analysis: refactoring changed enum with define to be able to use bitwise operators 2024-06-12 13:41:13 +02:00
temi54c1l8
6e009c0893 static analysis: cannot assign 'signed8' to different essential type 'boolean' [MISRA 2012 Rule 10.3, required] 2024-06-12 10:42:40 +02:00
temi54c1l8
9e8ceb4dfa static analysis: ignoring return value of functions [MISRA 2012 Directive 4.7, required] 2024-06-12 10:13:45 +02:00
temi54c1l8
d4e7214f4d static analysis: conditional of #if does not evaluate to 0 or 1 [MISRA 2012 Rule 20.8, required] 2024-06-12 09:09:12 +02:00
temi54c1l8
0142f41857 static analysis: an enum value is not an appropriate left operand [MISRA 2012 Rule 10.1, required] 2024-06-10 17:59:05 +02:00
temi54c1l8
446925be63 static analysis: add 'U' of unsigned [MISRA 2012 Rule 10.4, required] 2024-06-10 17:57:42 +02:00
temi54c1l8
dc10900ad5 static analysis: Aggiunte parentesi per la dipendenza posta sulla precedenza dell'operatore [MISRA 2012 Rule 12.1, advisory] 2024-06-10 14:39:05 +02:00
temi54c1l8
a17b9fa209 static analysis: Ignoring return value of functions: CO_setUint8, CO_setUint16, CO_setUint32 [MISRA 2012 Directive 4.7, required] 2024-06-10 12:07:41 +02:00
temi54c1l8
e3206f2c7f static analysis: Ignoring return value of function 'memset' [MISRA 2012 Directive 4.7, required] 2024-06-10 11:56:47 +02:00
temi54c1l8
ddaa8974f1 static analysis: Ignoring return value of function 'memcpy' [MISRA 2012 Directive 4.7, required] 2024-06-10 10:30:59 +02:00
Janez
6a0df55b01 Overrun not correctly reported in CO_Emergency.c, by H-Grobben #466 2024-06-04 19:54:00 +02:00
Janez
39fc9212e6 Fixed Compiler warnings, issue #512 2024-04-25 16:51:37 +02:00
Brian Linari
a52db428f6
Misra (#501)
Make MISRA compliant for:
MISRA C 2004 rule 14.10
MISRA C 2004 rule 14.9
MISRA C 2004 rule 13.1
MISRA C 2004 rule 15.3

MISRA style guild update
2024-03-13 17:20:42 +01:00
Janez
fe3777b781 Verify for restricted CAN-IDs in SYNC, TIME, EMCY, PDO and SDO COB-ID configuration. 2021-08-09 14:12:19 +02:00
Janez
b9927ff1cd Remove CO_CONFIG_EM_BUFFER_SIZE macro, size is fetched from Object Dictionary
It is necessary to re-generate OD.h file with the latest CANopenEditor
from https://github.com/CANopenNode/CANopenEditor
2021-06-28 12:39:03 +02:00
Janez
cdad71beee Fix reading OD entry 1003,0, etc. 2021-06-26 13:35:43 +02:00
Janez
1775b96450 Move emergency processing before NMT processing in CANopen.c
Hold emergency messages if not operational or pre-operational, don't
delete them.
There was a problem, if error condition was present after a
communication reset, device was in operational state for a short time.
2021-05-17 13:33:37 +02:00
gotocoffee1
591a91be29 fixed some warnings
fixed some small bugs
2021-04-27 16:24:00 +02:00
Janez
66d0e14ef4 301/CO_PDO.h/c updated to newOD. 2021-03-18 15:13:06 +01:00
Janez
926b59f3fd Fix missing CO_CONFIG_EM_PROD_INHIBIT switches 2021-03-15 11:58:20 +01:00
Janez
f8eb4bc0a1 CO_ODinterface: simplified API to read/write functions. Return variables changed. 2021-03-12 11:42:34 +01:00
Janez
5b8be4ef7e CO_ODinterface: simplified API to read/write functions. subIndex moved to OD_stream_t. 2021-03-10 15:22:03 +01:00
Janez
fc81777097 Pass CO_CANmodule_t argument to all the CO_LOCK_...() and CO_UNLOCK_...() macros.
https://github.com/CANopenNode/CANopenNode/issues/162#issuecomment-777438875
2021-03-02 21:28:22 +01:00