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

Transition to the fdcan-rs crate #39

Merged
merged 1 commit into from
Feb 15, 2023

Conversation

luctius
Copy link
Collaborator

@luctius luctius commented Dec 13, 2021

As discussed in #34, the FDCAN part has been transitioned to a new crate. This PR uses that crate instead of our own implementation.

@luctius luctius mentioned this pull request Dec 13, 2021
3 tasks
@kevswims
Copy link
Contributor

@luctius is there anything I can do to help get this in? I have a board with an STM32g4 and 2 CAN transceivers on it that I can do testing on if needed. At the moment I am using a super hacky driver in my own code to get the CAN working but I would love to move to this if possible.

@luctius luctius force-pushed the features/fdcan-rs branch 2 times, most recently from fe728d8 to 452437f Compare February 13, 2023 08:21
@luctius
Copy link
Collaborator Author

luctius commented Feb 13, 2023

I've updated this to fdcan 0.1.2. It builds; but if you have time I would mind if you could check that everything still works.

Copy link
Contributor

@kevswims kevswims left a comment

Choose a reason for hiding this comment

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

I made some changes to the example itself to get this going on my hardware and I was able to get the the two channels on my board to talk to each other. Functionally, I think this is good to go in.

I can put up a follow-on PR that cleans up the example to allow running it either on physical hardware or in loopback mode for testing. I should be able to get that up in a day or two.

Cargo.toml Outdated
@@ -23,6 +23,14 @@ static_assertions = "1.1"
version = "0.7.7"
features = ["critical-section-single-core"]

[dependencies.fdcan]
# git = "https://github.com/stm32-rs/fdcan.git"
Copy link
Contributor

Choose a reason for hiding this comment

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

Probably don't want to merge with these commented bits in here.

src/lib.rs Outdated
// pub mod can;
pub mod fdcan;
pub mod can;
// pub mod fdcan;
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove commented code?

@luctius luctius merged commit 621bd09 into stm32-rs:master Feb 15, 2023
@luctius luctius deleted the features/fdcan-rs branch March 21, 2023 06:59
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