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.

Sign up for GitHub

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

Jump to bottom

shell: provide shell commands to poll lsm303dlhc sensor #1945

Merged
merged 1 commit into from
Nov 6, 2014
Merged

shell: provide shell commands to poll lsm303dlhc sensor #1945

merged 1 commit into from
Nov 6, 2014

Conversation

thomaseichinger
Copy link
Member

Introduces shell commands lsm303dlhc_init and lsm303dlhc_read to interact with the lsm303dlhc driver.

@thomaseichinger thomaseichinger added Area: drivers Area: Device drivers Type: new feature The issue requests / The PR implemements a new feature for RIOT labels Nov 5, 2014
@thomaseichinger thomaseichinger added this to the Release NEXT MAJOR milestone Nov 5, 2014
@thomaseichinger
Copy link
Member Author

Will need adoption to #1946

@thomaseichinger thomaseichinger added the State: waiting for other PR State: The PR requires another PR to be merged first label Nov 5, 2014
@haukepetersen
Copy link
Contributor

Looks good. #1946 is merged, so ACK after you rebase

@haukepetersen haukepetersen removed the State: waiting for other PR State: The PR requires another PR to be merged first label Nov 5, 2014
@thomaseichinger
Copy link
Member Author

rebased and adopted changes from #1946

@thomaseichinger
Copy link
Member Author

rebased one last time after merge of #1949, waiting for Travis.

@haukepetersen
Copy link
Contributor

ACK and go.

haukepetersen added a commit that referenced this pull request Nov 6, 2014
@haukepetersen
Merge pull request #1945 from thomaseichinger/sc_lsm303dlhc
4cdc79f 
shell: provide shell commands to poll lsm303dlhc sensor
@haukepetersen haukepetersen merged commit 4cdc79f into RIOT-OS:master Nov 6, 2014
@LudwigKnuepfer
Copy link
Member

This should have been at least two commits I guess...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@haukepetersen haukepetersen

Labels
Area: drivers Area: Device drivers Type: new feature The issue requests / The PR implemements a new feature for RIOT
Projects
None yet
Milestone
Release 2014.12
Development

Successfully merging this pull request may close these issues.
3 participants
@thomaseichinger @haukepetersen @LudwigKnuepfer