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: enabled C++ for arduino-due and udoo #2031

Merged
merged 2 commits into from
Nov 18, 2014

Conversation

haukepetersen
Copy link
Contributor

2 more towards finishing the list in #1323

@haukepetersen haukepetersen added Platform: ARM Platform: This PR/issue effects ARM-based platforms Area: C++ Area: C++ wrapper Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation labels Nov 18, 2014
@haukepetersen haukepetersen added this to the Release NEXT MAJOR milestone Nov 18, 2014
@BytesGalore
Copy link
Member

nice, alternative to #1497

@haukepetersen
Copy link
Contributor Author

jupp, just saw #1497 -> the github search failed me. I am closing this in favor (of the original) #1497

@haukepetersen
Copy link
Contributor Author

Hm, something is wrong with my eyes today, the udoo is missing in #1497. I could just throw out everything that is not connected to the udoo board so both PRs make sense?

@BytesGalore
Copy link
Member

we just close #1497

@haukepetersen
Copy link
Contributor Author

if thats ok with you. Sorry for basically overwriting your work...

@BytesGalore
Copy link
Member

np, ACK for the arduino due part. (have no UDOO for testing)

@dangnhat
Copy link
Member

@haukepetersen : looks good to me. I would give an ACK but I don't have the hardware to test. :(
@BytesGalore : Do you have the hardware? If yes, then I will give you the honor to test and push the merge button. :-D

@BytesGalore
Copy link
Member

@haukepetersen @dangnhat, if someone could provide certainty on the udoo board is working it would be an honor for me to merge this PR :)

@haukepetersen
Copy link
Contributor Author

The UDOO is hardware wise identical to the arduino-due (even the pin mappings). The difference is that it comes with a quad-core Cortex-A9 co-processor and it is slightly different to flash... So I guess you can ack without trouble?!

@dangnhat dangnhat assigned BytesGalore and unassigned dangnhat Nov 18, 2014
BytesGalore pushed a commit that referenced this pull request Nov 18, 2014
@BytesGalore BytesGalore merged commit b5435fa into RIOT-OS:master Nov 18, 2014
@haukepetersen haukepetersen deleted the fix_sam3x_cpp branch November 18, 2014 17:41
@BytesGalore
Copy link
Member

@haukepetersen @dangnhat the merge message has disappeared,
So ACK and go! (just for the protocol)

@haukepetersen
Copy link
Contributor Author

nice, I updated the list in #1323

@BytesGalore
Copy link
Member

thx

@OlegHahm OlegHahm modified the milestone: Release 2015.12 Dec 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: C++ Area: C++ wrapper Platform: ARM Platform: This PR/issue effects ARM-based platforms Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants