Skip to content
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

I think FDCAN is missing? #127

Open
brandonros opened this issue Nov 7, 2022 · 3 comments
Open

I think FDCAN is missing? #127

brandonros opened this issue Nov 7, 2022 · 3 comments
Labels
help wanted Extra attention is needed

Comments

@brandonros
Copy link

Sorry, not trying to be annoying.

https://github.com/stm32-rs/stm32g4xx-hal/blob/16d6ae2a4b7d38805fbe614d6494ddd2a83429c8/src/fdcan.rs

just not sure if that is supposed to be ported over to this repo?

@mattico
Copy link
Contributor

mattico commented Nov 8, 2022

Yep! There are many things missing. The device family HALs are mostly separate projects run by mostly separate people. If you're looking to add FDCAN support, copying the most complete version from the most similar device can be a good place to start.

The stm32-rs/stm32-rs data files can help to find which devices have similar peripherals, or the peripheral map from embassy-rs though it looks like they haven't implemented FDCAN yet. And in this case the STM32G4's FDCAN registers are probably very similar.

Even if the G4's FDCAN peripheral is identical to the G0 peripheral there are usually different bugs in ST's SVD files, which are used to generate the register access HAL, which will need to be fixed.

@brandonros
Copy link
Author

Appreciate the insight.

Not to add more fire to the flame but it seems that… the fdcan driver itself is in the process of being copied/moved out of the stm32gfxxx repos and moved into its own at https://github.com/stm32-rs/fdcan… maybe? I’m not sure

Then that was was supposed to implement embedded-can traits for embedded-hal, but that’s commented out (just putting that here for anybody else that tries to go down this rabbit hole)

I’m sure the community will catch up/clean up support for these chips soon enough, thank you for everything this team does for the community!

@mattico
Copy link
Contributor

mattico commented Nov 10, 2022

Didn't intend to sound testy, just wanted to add some info for if you needed to get FDCAN working.

@dotcypress dotcypress added the help wanted Extra attention is needed label Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants