Skip to content

CAN support with bxcan crate #352

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

Merged
merged 6 commits into from
Aug 18, 2021
Merged

CAN support with bxcan crate #352

merged 6 commits into from
Aug 18, 2021

Conversation

timokroeger
Copy link
Contributor

@timokroeger timokroeger commented Aug 5, 2021

Ready to review

Example code not working yet.
stm32-rs/stm32f1xx-hal#215 (comment) might be related

@timokroeger timokroeger marked this pull request as ready for review August 14, 2021 21:15
@timokroeger
Copy link
Contributor Author

timokroeger commented Aug 14, 2021

CI is red because embassy-rs/stm32-data#74 is required

@Dirbaio
Copy link
Member

Dirbaio commented Aug 14, 2021

Merged! Could you bump the submodule to get green? Thanks :)

Copy link
Member

@Dirbaio Dirbaio left a comment

Choose a reason for hiding this comment

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

Really clean and nice, thank you! 🚀

There seems no way to enable a optional dependency from build.rs or
features passed through the command line.
Older families like F1 and F4 have a consistent naming for the CAN
peripherals: CAN when there is only one instance, CAN1/CAN2/CAN2 if
there are multiple instances.
Newer families like L4 and F7 use the name CAN1 even if there is only
one instance. The number of filter banks is incorrect for those.

Affected chips:
* STM32F722
* STM32F723
* STM32F730
* STM32F732
* STM32F733
* STM32L4P5
* STM32L4Q5
* STM32L4R5
* STM32L4R7
* STM32L4R9
* STM32L4S5
* STM32L4S7
* STM32L4S9
* STM32L431
* STM32L432
* STM32L433
* STM32L442
* STM32L443
* STM32L451
* STM32L452
* STM32L462
* STM32L471
* STM32L475
* STM32L476
* STM32L485
* STM32L486
@Dirbaio Dirbaio merged commit 2b5d1c0 into embassy-rs:master Aug 18, 2021
@timokroeger timokroeger deleted the can branch August 26, 2022 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants