nikon/CANopenNode
1
0
Fork 0
Code Activity Actions
792 commits 3 branches 8 tags 5.5 MiB
Commit graph

81 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
Tilen Majerle
6dfd4ed7f2
define variable before it is being assigned (#582) 2025-06-08 17:43:26 +02:00
Janez
f46f1e2733 clang-format: ReflowComments: true 2024-07-07 01:05:10 +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
Janez
7db21c1f91 CO_SDO_server.c fix missing "bufShifted" 2024-07-05 11:14:37 +02:00
temi54c1l8
5cf08656e1 Fix commit 29bb7992d2 won't respond with abort message, if segmented transfer is not supported 2024-06-26 08:50:45 +02:00
temi54c1l8
f7c7f9c104 Add more static analysis and MISRA.md 
(cherry picked from commit ad337c3e9c4e879721ec432bc9a5d9a84a53c573)

# Conflicts:
#	CANopen.c
 2024-06-24 17:46:34 +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
d2770c4917 CO_SDOserver: static analysis: named parameter 'calculateCrc' of 'non-virtual' function 'readFromOd' not subsequently referenced [MISRA 2012 Rule 2.7, advisory] 2024-06-14 11:15:41 +02:00
temi54c1l8
29bb7992d2 CO_SDOserver: static analysis: enum constants not used within default switch 2024-06-14 09:49:21 +02:00
temi54c1l8
cbf3df74e9 CO_SDOserver: static analysis: result of assignment operator used in right operand to '=' operator [MISRA 2012 Rule 13.4, advisory] 2024-06-14 09:47:16 +02:00
temi54c1l8
06834c1d14 O_SDOserver: static analysis: side effects on right hand of logical operator, '&&' [MISRA 2012 Rule 13.5, required] 
because SDO->state is volatile
 2024-06-14 09:02:17 +02:00
temi54c1l8
35cefe09c3 CO_SDOserver: static analysis: cannot cast 'unsigned8' to wider essential type 'unsigned32' [MISRA 2012 Rule 10.8, required] 2024-06-14 08:52:07 +02:00
temi54c1l8
8af3ef3c6b CO_SDOserver: static analysis: taking address of array 2024-06-14 08:26:34 +02:00
temi54c1l8
99b56d10c9 CO_SDOserver: static analysis: unexpected lack of indentation 2024-06-13 13:21:20 +02:00
temi54c1l8
e82c942846 Fix previous static analysis 2024-06-13 13:18:02 +02:00
temi54c1l8
00f8961f55 CO_SDOserver: static analysis: loss of precision (assignment) from 32 bits to 8 bits 2024-06-13 13:16:22 +02:00
temi54c1l8
404ebca5fb CO_SDOserver: 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 13:14:00 +02:00
temi54c1l8
a4fb95e38b CO_SDOserver: static analysis: statement or comment should appear in default case [MISRA 2012 Rule 16.1, required], [MISRA 2012 Rule 16.4, required] 2024-06-13 13:11:09 +02:00
temi54c1l8
b2294a65ab CO_SDOserver: static analysis: argument 1 of type 'uint8_t *' (aka 'unsigned char *') is not compatible with argument 2 of type 'uint32_t *' (aka 'unsigned int *') in call to function 'memcpy' 2024-06-13 13:09:34 +02:00
temi54c1l8
460441f972 CO_SDOserve: static analysis: increment/decrement operation combined with other operation with side-effects [MISRA 2012 Rule 13.3, advisory] 2024-06-13 12:29:43 +02:00
temi54c1l8
f8d10390bc CO_SDOserver: static analysis: missing unconditional break from final switch case [MISRA 2012 Rule 16.1, required], [MISRA 2012 Rule 16.3, required] 2024-06-13 12:27:50 +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
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
187650d073 static analysis: Ignoring return value of function 'memmove' [MISRA 2012 Directive 4.7, required] 2024-06-10 11:58:55 +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
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
mlout
4ab0bd10c1 Always call OD lock macros on SDO operations 2023-03-17 17:42:35 +01:00
mlout
900ab00a79 Rename CO_CONFIG_FLAG_ALWAYS_LOCK_OD to CO_CONFIG_FLAG_SDO_ALWAYS_LOCK_OD 2023-03-16 14:19:07 +01:00
mlout
d8a637dbc5 Add CO_CONFIG_FLAG_ALWAYS_LOCK_OD to always lock OD 
Add CO_CONFIG_FLAG_ALWAYS_LOCK_OD which can be used to lock the OD for
every SDO read/write operation.
Modify the code in CO_SDOserver.c and CO_SDOclient.c to check this
config flag.
 2023-03-16 14:09:01 +01:00
ankit.chudasama1
4c86a61342 Fix variable name and use of pointer for big endian system 
1. Variable name update in CO_PDO.c file
2. Correct use of pointer in the CO_SDOserver.c file
 2023-02-16 19:36:49 +05:30
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
bb74ed2cf5 SDO server: send abort on wrong block size 2021-08-09 14:00:45 +02:00
Janez
b4ddf9be32 Fixed SDO server 
- Missing break
- expedited transfer: always copied fixed amount of data instead of actual data size.
 2021-08-09 13:56:05 +02:00
gotocoffee1
591a91be29 fixed some warnings 
fixed some small bugs
 2021-04-27 16:24:00 +02:00
Julien PEYREGNE
877edc50a8 Reduce variable scope 2021-03-19 09:54:17 +01:00
Julien PEYREGNE
b6f04d65f8 Fix null pointer dereference warnings 2021-03-19 09:54:17 +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