Skip to content

mulle: Initialize xtimer early to support nvram usage in board_init#4133

Merged
PeterKietzmann merged 1 commit intoRIOT-OS:masterfrom
jnohlgard:pr/mulle-xtimer-init
Oct 22, 2015
Merged

mulle: Initialize xtimer early to support nvram usage in board_init#4133
PeterKietzmann merged 1 commit intoRIOT-OS:masterfrom
jnohlgard:pr/mulle-xtimer-init

Conversation

@jnohlgard
Copy link
Copy Markdown
Member

xtimer_init is normally run by auto_init, but that runs after board_init and we want to use nvram_spi inside board_init. Mulle is currently broken in master, error introduced by me in #3485 (there used to be a hwtimer_init inside board_init before, but it went away during some rebase)

Tested working on mulle.

@jnohlgard jnohlgard added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Platform: ARM Platform: This PR/issue effects ARM-based platforms Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Oct 22, 2015
@PeterKietzmann
Copy link
Copy Markdown
Member

Seems reasonable. I need to trust you in that. ACK and go

PeterKietzmann added a commit that referenced this pull request Oct 22, 2015
mulle: Initialize xtimer early to support nvram usage in board_init
@PeterKietzmann PeterKietzmann merged commit c59f245 into RIOT-OS:master Oct 22, 2015
@jnohlgard jnohlgard deleted the pr/mulle-xtimer-init branch April 6, 2017 08:14
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 Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Platform: ARM Platform: This PR/issue effects ARM-based platforms Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants