Skip to content

native: peripheral random interface#2041

Merged
LudwigKnuepfer merged 3 commits intoRIOT-OS:masterfrom
LudwigKnuepfer:native-random
Nov 19, 2014
Merged

native: peripheral random interface#2041
LudwigKnuepfer merged 3 commits intoRIOT-OS:masterfrom
LudwigKnuepfer:native-random

Conversation

@LudwigKnuepfer
Copy link
Copy Markdown
Member

  • add native implementation of drivers/periph/random.h
  • fix some warning and coding conventions in tests/periph_random

@LudwigKnuepfer LudwigKnuepfer added Platform: native Platform: This PR/issue effects the native platform Type: new feature The issue requests / The PR implemements a new feature for RIOT labels Nov 19, 2014
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

you could save 1 newline here...

@haukepetersen
Copy link
Copy Markdown
Contributor

two minor style comments. Tested and works fine! ACK when above comments are addressed and Travis is happy

@LudwigKnuepfer LudwigKnuepfer added the CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable label Nov 19, 2014
@LudwigKnuepfer LudwigKnuepfer removed the CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable label Nov 19, 2014
@LudwigKnuepfer
Copy link
Copy Markdown
Member Author

Travis failed for unrelated reasons.

LudwigKnuepfer pushed a commit that referenced this pull request Nov 19, 2014
native: peripheral random interface
@LudwigKnuepfer LudwigKnuepfer merged commit 6eaaac6 into RIOT-OS:master Nov 19, 2014
@LudwigKnuepfer LudwigKnuepfer deleted the native-random branch November 19, 2014 14:25
@OlegHahm OlegHahm mentioned this pull request Nov 23, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Platform: native Platform: This PR/issue effects the native platform Type: new feature The issue requests / The PR implemements a new feature for RIOT

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants