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

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

Merged
merged 1 commit into from
Oct 22, 2015

Conversation

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