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

Add support for Adafruit CANBus Feather RP2040 #36

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ThadHouse
Copy link
Contributor

@ThadHouse ThadHouse commented Aug 12, 2023

Adds support for https://www.adafruit.com/product/5724
Copied mostly from the adafruit-feather-rp2040

@ThadHouse ThadHouse force-pushed the adafruit-canbus-feather-rp2040 branch 3 times, most recently from d8c1fcd to 7a8f7b8 Compare October 13, 2023 01:12
Adds support for https://www.adafruit.com/product/5724
Copied mostly from the adafruit-feather-rp2040
@ThadHouse ThadHouse force-pushed the adafruit-canbus-feather-rp2040 branch from 7a8f7b8 to 3f4b770 Compare October 13, 2023 16:40
Copy link
Member

@ithinuel ithinuel left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution, this is greatly appreciated!

Similarly to #45 I think it'd be better having board relevant examples rather than generic that already exist for several of other boards in this repository.

This board has a special purpose revolving around CAN, why not building one that sends/receives some frame on the CAN bus and update the neopixel's state accordingly?

@ThadHouse
Copy link
Contributor Author

Can an example be done in a later PR? From what I've dug into, there's no great MCP2515 CAN Library for Rust, all the ones that exist are pretty iffy. Doesn't make sense to leave these PR's in PR Limbo on examples, when the pin layouts are still worth it to have.

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