Skip to content

cpu/stm32: unify cpu_init()#8735

Merged
haukepetersen merged 1 commit intoRIOT-OS:masterfrom
kaspar030:stm32_unify_cpu_init
Mar 6, 2018
Merged

cpu/stm32: unify cpu_init()#8735
haukepetersen merged 1 commit intoRIOT-OS:masterfrom
kaspar030:stm32_unify_cpu_init

Conversation

@kaspar030
Copy link
Copy Markdown
Contributor

Contribution description

stm32's cpu.c look the same on all stm32fx and stm32l4, and almost identical for stm32l0&1.

This PR unifies accordingly.

@kaspar030 kaspar030 added Type: cleanup The issue proposes a clean-up / The PR cleans-up 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 Mar 5, 2018
@kaspar030 kaspar030 force-pushed the stm32_unify_cpu_init branch from 80edbc0 to 3b42fb4 Compare March 5, 2018 11:35
@aabadie
Copy link
Copy Markdown
Contributor

aabadie commented Mar 5, 2018

Very nice !

@aabadie
Copy link
Copy Markdown
Contributor

aabadie commented Mar 5, 2018

stm32's cpu.c look the same on all stm32fx and stm32l4

Looking at the code, the stmclk.c of stm32l4 hasn't been merged with other stm32fx. Is there something I missed ?

@aabadie
Copy link
Copy Markdown
Contributor

aabadie commented Mar 5, 2018

Just to keep track of the testing on cpu family boards:

  • stm32l0: works on b-l072z-lrwan1
  • stm32l1: works on nucleo-l152 (rec C-03)
  • stm32l4: works on b-l475e-iot01a
  • stm32f0
  • stm32f1 (iotlab-m3)
  • stm32f2
  • stm32f3
  • stm32f4
  • stm32f7

Copy link
Copy Markdown
Contributor

@aabadie aabadie left a comment

Choose a reason for hiding this comment

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

Tested on all stm32 cpu families, all worked. I have no comment on the code changes, they are all fine.

ACK

@aabadie
Copy link
Copy Markdown
Contributor

aabadie commented Mar 6, 2018

I let @haukepetersen a chance to review this, just in case.

@kaspar030
Copy link
Copy Markdown
Contributor Author

Looking at the code, the stmclk.c of stm32l4 hasn't been merged with other stm32fx. Is there something I missed ?

True, I missed that file. But it seems to have quite some differences, thus it'll have to go folo.

Copy link
Copy Markdown
Contributor

@haukepetersen haukepetersen left a comment

Choose a reason for hiding this comment

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

ACK -> nice one!

@haukepetersen
Copy link
Copy Markdown
Contributor

all green -> go go go

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