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

CAN trait implementation #72

Open
marcelbuesing opened this issue Dec 9, 2021 · 8 comments
Open

CAN trait implementation #72

marcelbuesing opened this issue Dec 9, 2021 · 8 comments
Labels
enhancement New feature or request

Comments

@marcelbuesing
Copy link

I think it would be great to add implementations for the new embedded-hal CAN traits via socketcan. Would you be open for a PR for this?

@eldruin
Copy link
Member

eldruin commented Dec 9, 2021

That would definitely be interesting!

@ryankurte ryankurte added the enhancement New feature or request label Dec 9, 2021
@marcelbuesing
Copy link
Author

Ok I basically created a fork of the socketcan crate, you can find the fork here. Sadly the original crate is no longer maintained but works quite well for me and still sees significant download numbers on crates.io. I reduced the dependencies and removed modules which were not necessary for this use case. A simple example is here. It already works but still needs some minor cleanup.

My suggestion would be to move this project/fork copy to the rust-embedded organization and basically move all the code that is currently in the embedded_hal_impl module to the linux-embedded-hal crate referencing the socketcan crate in the rust-embedded organization, analogous to gpio-cdev or sysfs_gpio. If this an option for the rust-embedded org would this be ok for your @mbr ? I see great potential in e.g. getting started with the embedded-hal socketcan implementation and then e.g. migrating to bxcan in projects.

@ryankurte
Copy link
Contributor

ryankurte commented Dec 14, 2021

hey thanks for your effort! seems like a fundamental crate so if @mbr (cc. @fpagliughi) are up for handing socketcan over it's a 👍 from me, any other opinions @rust-embedded/tools @rust-embedded/embedded-linux (whoops)?

@adamgreig
Copy link
Member

(maybe @rust-embedded/embedded-linux ?)

@nastevens
Copy link
Member

Seems logical to me as well 👍

@marcelbuesing
Copy link
Author

marcelbuesing commented Feb 10, 2022

Ok since there is no response from the socketcan maintainers, how about a new crate name: linux-socketcan?

@fpagliughi
Copy link

fpagliughi commented Feb 10, 2022

Hey All. Apologies. I agreed to share maintainer responsibilities for the socketcan crate last year, then got pretty sick, missed a few months of work, and let my opensource projects go dormant for a while. But I've been catching up recently, and have a CAN project starting soon, so can begin contributing.

@fpagliughi
Copy link

fpagliughi commented Feb 10, 2022

BTW... I'll have to track down @mbr. I believe he gave me write access to the GitHub repo, but not crates.io...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

6 participants