michaelh/zephyr
1
0
Fork 0
Code Releases Activity

drivers: can: fix can_configure() when CAN-FD is enabled

Browse source 
Currently, can_configure() pass a hard-coded 0 for the data bitrate
(which is only used for CAN-FD), breaking this API for CAN-FD enabled
applications.

Instead pass in the provided bitrate for both arbitration phase and data
phase.

Fixes: #34375

Signed-off-by: Henrik Brix Andersen <hebad@vestas.com>
This commit is contained in:
Henrik Brix Andersen 2021-12-01 17:36:09 +01:00 • committed by Carles Cufí
commit 8d5a0664d2
1 changed files with 1 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

3
include/drivers/can.h
View file

@ -860,7 +860,7 @@ __deprecated static inline int can_configure(const struct device *dev, enum can_
uint32_t bitrate) uint32_t bitrate)
{ {
if (bitrate > 0) { if (bitrate > 0) {
int err = can_set_bitrate(dev, bitrate, 0); int err = can_set_bitrate(dev, bitrate, bitrate);
if (err != 0) { if (err != 0) {
return err; return err;
} }
@ -869,7 +869,6 @@ __deprecated static inline int can_configure(const struct device *dev, enum can_
return can_set_mode(dev, mode); return can_set_mode(dev, mode);
} }
/** /**
* @brief Get current state * @brief Get current state
* *