-
Notifications
You must be signed in to change notification settings - Fork 179
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support for CAN peripherals with the bxcan
crate
#293
Conversation
bxcan
cratebxcan
crate
e290162
to
a377cf4
Compare
Any news? |
bxcan
cratebxcan
crate
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Finally found the time to review this, thanks for all the effort you've put into this.
The changes look good, there are a few compiler warnings and a missing changelog entry, but other than that, it looks ready to merge!
bxcan 0.4.0 has been released, you might want to update to that before merging |
Any news? |
Finally got around to looking at this, sorry about the delay! Everything looks great, thanks everyone who has helped out getting this working 😍 |
Thanks for the work! Any plans on release date for can support? |
Replaces #215
@jonas-schievink refactored the previous PR into a device agnostic driver crate.
Use the new crate to minimize fragmentation between
stm32fxxx-hal
crates.Draft until I have upstreamed a last missing improvement to masked filters.