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/kinetis+boards: fixed UART configuration #6958

Merged
merged 1 commit into from
Apr 25, 2017
Merged

cpu/kinetis+boards: fixed UART configuration #6958

merged 1 commit into from
Apr 25, 2017

Conversation

haukepetersen
Copy link
Contributor

In #6868 I overlooked a sub-optimal UART configuration. This PR enables to arbitrarily order the (logical) UART devices for Kinetis CPUs (as done for other platforms), so no need to configure unused UART devices anymore -> saves ROM on e.g. phyNODE

tested on mulle and phyNODE

@haukepetersen haukepetersen added 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 CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Apr 25, 2017
@haukepetersen haukepetersen added this to the Release 2017.07 milestone Apr 25, 2017
@haukepetersen haukepetersen requested a review from jnohlgard April 25, 2017 12:30
jnohlgard
jnohlgard approved these changes Apr 25, 2017
View reviewed changes
Copy link
Member

@jnohlgard jnohlgard 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

@miri64 miri64 merged commit 0cc1595 into RIOT-OS:master Apr 25, 2017
@haukepetersen haukepetersen deleted the fix_kinetis_uart branch April 25, 2017 15:41
@jnohlgard jnohlgard changed the title cpu/kintis+boards: fixed UART configuration cpu/kinetis+boards: fixed UART configuration Apr 28, 2017
@jnohlgard jnohlgard mentioned this pull request Apr 28, 2017
Closed
@kYc0o kYc0o mentioned this pull request May 1, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mfrey mfrey mfrey approved these changes

@jnohlgard jnohlgard jnohlgard approved these changes

Assignees

@mfrey mfrey

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: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Milestone
Release 2017.07
Development

Successfully merging this pull request may close these issues.
4 participants
@haukepetersen @mfrey @jnohlgard @miri64