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