Skip to content

Conversation

@congnguyenhuu
Copy link
Contributor

Reuse existing MCUX-based shim driver for FlexCAN and update to compatible with the hardware block in S32K344. Each CAN instance of S32K344 has different maximum number of message buffers, depends on payload. Add kconfig that defines maximum number of message buffers for concurrent active instances and update driver to compatible.

@zephyrbot
Copy link

zephyrbot commented Jul 10, 2023

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
hal_nxp zephyrproject-rtos/hal_nxp@35b0e5a zephyrproject-rtos/hal_nxp@9fb917f (master) zephyrproject-rtos/[email protected]

Note: This message is automatically posted and updated by the Manifest GitHub Action.

Copy link
Member

@henrikbrixandersen henrikbrixandersen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Please get the NXP HAL PR merged so we can get this in.

@mmahadevan108
Copy link
Contributor

@congnguyenhuu , can you please update the west.yml file as the HAL PR has been merged.

Each CAN instance of S32K344 has different maximum number
of message buffers, depends on payload. Add kconfig that
defines maximum number of message buffers for concurrent
active instances and update driver to compatible
support S32k344.

Signed-off-by: Cong Nguyen Huu <[email protected]>
Reuse existing MCUX-based shim driver for FlexCAN.
Enable flexcan0 for Zephyr canbus to run tests.

Signed-off-by: Cong Nguyen Huu <[email protected]>
@zephyrbot zephyrbot removed the DNM This PR should not be merged (Do Not Merge) label Jul 26, 2023
@mmahadevan108 mmahadevan108 merged commit 36d63e1 into zephyrproject-rtos:main Jul 27, 2023
@manuargue manuargue deleted the mr_canhubk3_flexcan branch July 27, 2023 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants