Kwabena/mimxrt can#142
Open
kwagyeman wants to merge 1 commit into
Open
Conversation
Member
|
Could you squash this in one commit? I'll drop it when rebasing. Also, is there a corresponding openmv pr or just needs a submodule update? |
Adds a CAN driver for the mimxrt port using the new machine.CAN API, including: - FlexCAN driver with TX queue, RX FIFO, filters, IRQ handling, error/state reporting, and cancel_send() per RM 40.7.7.1. - LOOPBACK, SILENT_LOOPBACK and SILENT modes. - CAN pin settings for all mimxrt boards, including OPENMV_RT1060. - extmod/machine_can: preserve the raw bitrate for port-specific init and add a callback to signal that all filters have been processed, with a matching stub for the stm32 port. - tests: add a mimxrt CAN target wiring configuration, align the CAN tests with the mimxrt port, and fix a startup race in machine_can_05_tx_prio_cancel.py. Co-authored-by: Kwabena W. Agyeman <kwagyeman@live.com>
kwagyeman
force-pushed
the
kwabena/mimxrt_can
branch
from
405c561 to
33bb5d8
Compare
Member
Author
|
Squashed. And yes, there's an upstream PR: openmv/openmv#3187. It adds the deinits required. Otherwise you get a crash after soft reset. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds support for machine.CAN with the RT1060. Tested and works on the OpenMV Cam RT1062.