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

kinetis: update vendor headers #7434

Merged
merged 6 commits into from
Aug 3, 2017

Conversation

jnohlgard
Copy link
Member

based on #7379
This updates the vendor headers to the latest ones currently available in the tool chain packages from https://mcuxpresso.nxp.com
Moved all compatibility definitions to a separate header to make updating easier.

@jnohlgard jnohlgard added 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 Aug 1, 2017
@jnohlgard jnohlgard added this to the Release 2017.10 milestone Aug 1, 2017
@jnohlgard jnohlgard force-pushed the pr/kinetis-update-headers branch 2 times, most recently from 8e33bf0 to 6a877cc Compare August 1, 2017 14:45
@kYc0o
Copy link
Contributor

kYc0o commented Aug 1, 2017

Is this updated to master? I'd like to test the new headers with the new clocking configurations.

@jnohlgard
Copy link
Member Author

this should be on the latest master with the unified kinetis cpu.c and clocking

@kYc0o kYc0o added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Aug 2, 2017
@kYc0o
Copy link
Contributor

kYc0o commented Aug 2, 2017

Hmmm... Murdock fails because whitespaces in some lines... I thought these files were omitted isn't it?

@jnohlgard
Copy link
Member Author

I think they are omitted from style checks but not whitespace. I'll fix them and amend the PR when I am back at the computer

@jnohlgard jnohlgard force-pushed the pr/kinetis-update-headers branch from 6a877cc to 0ef0f7b Compare August 3, 2017 19:19
@jnohlgard
Copy link
Member Author

amended the whitespace change (deleted empty lines at the bottom of each header)

Copy link
Contributor

@kYc0o kYc0o left a comment

Choose a reason for hiding this comment

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

ACK.

@kYc0o kYc0o merged commit 0cb5cc7 into RIOT-OS:master Aug 3, 2017
@jnohlgard jnohlgard deleted the pr/kinetis-update-headers branch August 5, 2017 06:28
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: 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants