Skip to content

kinetis: update vendor headers#7434

Merged
kYc0o merged 6 commits intoRIOT-OS:masterfrom
jnohlgard:pr/kinetis-update-headers
Aug 3, 2017
Merged

kinetis: update vendor headers#7434
kYc0o merged 6 commits intoRIOT-OS:masterfrom
jnohlgard:pr/kinetis-update-headers

Conversation

@jnohlgard
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member Author

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

Copy link
Copy Markdown
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