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

boards/arduino-mega2560: cleanup in periph_conf.h #5021

Merged
merged 1 commit into from
Mar 10, 2016

Conversation

haukepetersen
Copy link
Contributor

rebased on #5020

removed some unused defines and cleaned up doxygen.

@haukepetersen haukepetersen added Platform: AVR Platform: This PR/issue effects AVR-based platforms Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation labels Mar 9, 2016
@haukepetersen haukepetersen added this to the Release 2016.04 milestone Mar 9, 2016
@OlegHahm
Copy link
Member

OlegHahm commented Mar 9, 2016

What about the added SPI?

@kYc0o
Copy link
Contributor

kYc0o commented Mar 9, 2016

@OlegHahm I'll test it ASAP ;)

@OlegHahm
Copy link
Member

OlegHahm commented Mar 9, 2016

I was more referring to the fact that this is not mentioned in the PR title or description.

@OlegHahm
Copy link
Member

OlegHahm commented Mar 9, 2016

Ah, now I get it. Overlooked the #5020 in the description, because the label was not set.

@OlegHahm OlegHahm added the State: waiting for other PR State: The PR requires another PR to be merged first label Mar 9, 2016
@kYc0o
Copy link
Contributor

kYc0o commented Mar 9, 2016

Oh yes, there are two different PRs

- remove unused periph configuration
- fixed doxygen
@haukepetersen haukepetersen force-pushed the fix_arduinomega_periphconf branch from f2b1221 to e55ea08 Compare March 10, 2016 13:04
@haukepetersen haukepetersen added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed State: waiting for other PR State: The PR requires another PR to be merged first labels Mar 10, 2016
@haukepetersen
Copy link
Contributor Author

dependencies are in, ready for review.

@kYc0o
Copy link
Contributor

kYc0o commented Mar 10, 2016

ACK and go

kYc0o added a commit that referenced this pull request Mar 10, 2016
boards/arduino-mega2560: cleanup in periph_conf.h
@kYc0o kYc0o merged commit 23a3b0e into RIOT-OS:master Mar 10, 2016
@haukepetersen haukepetersen deleted the fix_arduinomega_periphconf branch March 10, 2016 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: AVR Platform: This PR/issue effects AVR-based platforms Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants