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.

Sign up for GitHub

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

Jump to bottom

cpu/sam3: cleaned up UART driver #4170

Merged
merged 3 commits into from
Nov 11, 2015
Merged

cpu/sam3: cleaned up UART driver #4170

merged 3 commits into from
Nov 11, 2015

Conversation

haukepetersen
Copy link
Contributor

based on #4114

Cleaned up/optimized the UART driver for the sam3x and adjusted the arduino-due and the udoo boards

@haukepetersen haukepetersen added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Platform: ARM Platform: This PR/issue effects ARM-based platforms Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation labels Oct 27, 2015
@haukepetersen haukepetersen added this to the Release 2015.12 milestone Oct 27, 2015
@haukepetersen
Copy link
Contributor Author

rebased

@A-Paul
Copy link
Member

A-Paul commented Nov 5, 2015

Is a HW runtime check advised?

@A-Paul A-Paul added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Nov 5, 2015
@PeterKietzmann
Copy link
Member

@A-Paul yes please. The periph_uart test might help you

@A-Paul
Copy link
Member

A-Paul commented Nov 11, 2015

Tests on arduino-due with UART{1,2,3} and {9600,115200} baud showed no issues.

@A-Paul
Copy link
Member

A-Paul commented Nov 11, 2015

@haukepetersen ACK.

A-Paul pushed a commit that referenced this pull request Nov 11, 2015
Merge pull request #4170 from haukepetersen/opt_sam3_uart
e5b6566 
cpu/sam3: cleaned up UART driver
@A-Paul A-Paul merged commit e5b6566 into RIOT-OS:master Nov 11, 2015
@haukepetersen haukepetersen deleted the opt_sam3_uart branch November 18, 2015 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@A-Paul A-Paul

Labels
CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Milestone
Release 2015.12
Development

Successfully merging this pull request may close these issues.
3 participants
@haukepetersen @A-Paul @PeterKietzmann