Skip to content

cpu/stm32f1: udpated vendor headers#4681

Merged
miri64 merged 5 commits intoRIOT-OS:masterfrom
haukepetersen:opt_f1_cpuheaders
Jan 27, 2016
Merged

cpu/stm32f1: udpated vendor headers#4681
miri64 merged 5 commits intoRIOT-OS:masterfrom
haukepetersen:opt_f1_cpuheaders

Conversation

@haukepetersen
Copy link
Copy Markdown
Contributor

@haukepetersen haukepetersen commented Jan 25, 2016

The vendor header we were using for the stm32f1 is veeery old, it is from STM's pre-Cube-lib-days... The modern headers are much easier to read and more consistent with other STM headers in RIOT.

Also this should fix #392 for the stm32f1.

@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 Community: Hack'n'ACK candidate This PR is a candidate for review and discussion during one of RIOT's monthly Hack'n'ACK parties CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jan 25, 2016
@haukepetersen haukepetersen added this to the Release 2016.03 milestone Jan 25, 2016
@haukepetersen haukepetersen mentioned this pull request Jan 25, 2016
Closed
@haukepetersen
Copy link
Copy Markdown
Contributor Author

haukepetersen commented Jan 26, 2016

changed dos to unix line endings in header files and squashed

@thomaseichinger
Copy link
Copy Markdown
Member

thomaseichinger commented Jan 26, 2016

Changes look good to me. ACK when Travis passed.

@miri64
Copy link
Copy Markdown
Member

miri64 commented Jan 26, 2016

Needs rebase due to #3309

@haukepetersen
Copy link
Copy Markdown
Contributor Author

haukepetersen commented Jan 26, 2016

rebased

miri64 added a commit that referenced this pull request Jan 27, 2016
@miri64
Merge pull request #4681 from haukepetersen/opt_f1_cpuheaders
80023f0 
cpu/stm32f1: udpated vendor headers
@miri64 miri64 merged commit 80023f0 into RIOT-OS:master Jan 27, 2016
@haukepetersen haukepetersen deleted the opt_f1_cpuheaders branch February 5, 2016 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@thomaseichinger thomaseichinger

Labels

CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Community: Hack'n'ACK candidate This PR is a candidate for review and discussion during one of RIOT's monthly Hack'n'ACK parties 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 2016.04

Development

Successfully merging this pull request may close these issues.

clean up license headers

3 participants

@haukepetersen @thomaseichinger @miri64