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

arduino-hal: Add support for Arduino Mega #140

Merged
merged 9 commits into from
Feb 17, 2021

Conversation

explicite
Copy link
Contributor

Arduino support like in: 5304fd0

@explicite explicite changed the base branch from master to next February 16, 2021 20:46
@explicite
Copy link
Contributor Author

@Rahix compiled with cargo build --features arduino-mega in correct branch ;)

@Rahix Rahix self-requested a review February 17, 2021 07:48
@Rahix Rahix added the mcu-support Support for a new Microcontroller label Feb 17, 2021
Copy link
Owner

@Rahix Rahix left a comment

Choose a reason for hiding this comment

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

Thanks, looking really good already! I've got a few minor nits, apart from that I'm happy :) I'll do a test-build myself later today as CI doesn't cover this code yet without an examples crate, but I guess with that we're good to go.

Thanks again for helping with this work, I'm quite enjoying having you on board :)

arduino-hal/Cargo.toml Outdated Show resolved Hide resolved
arduino-hal/src/clock.rs Outdated Show resolved Hide resolved
@Rahix Rahix added this to the New APIs milestone Feb 17, 2021
- split cloc impl
Copy link
Owner

@Rahix Rahix left a comment

Choose a reason for hiding this comment

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

Looks good, thanks a lot!

@Rahix Rahix merged commit 9e41a1e into Rahix:next Feb 17, 2021
@explicite explicite deleted the arduino-hal/atmega-2560 branch February 20, 2021 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mcu-support Support for a new Microcontroller
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants