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: sensors: exit on failure #1952

Merged
merged 1 commit into from
Nov 6, 2014
Merged

shell: sensors: exit on failure #1952

merged 1 commit into from
Nov 6, 2014

Conversation

OlegHahm
Copy link
Member

@OlegHahm OlegHahm commented Nov 6, 2014

Immediately leave the read command if sensor is not initialized.

Otherwise the board just hangs after the error message.

@OlegHahm
shell: sensors: exit on failure
f79e228 
Immediately leave the read command if sensor is not initialized.
@OlegHahm OlegHahm added 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) Area: drivers Area: Device drivers Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer and removed Platform: ARM Platform: This PR/issue effects ARM-based platforms labels Nov 6, 2014
@OlegHahm OlegHahm added this to the Release NEXT MAJOR milestone Nov 6, 2014
@thomaseichinger
Copy link
Member

ACK & Go

thomaseichinger added a commit that referenced this pull request Nov 6, 2014
@thomaseichinger
Merge pull request #1952 from OlegHahm/sc_iotlab_sensors_check
f6175b7 
shell: sensors: exit on failure
@thomaseichinger thomaseichinger merged commit f6175b7 into RIOT-OS:master Nov 6, 2014
@OlegHahm OlegHahm deleted the sc_iotlab_sensors_check branch November 6, 2014 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@thomaseichinger thomaseichinger

Labels
Area: drivers Area: Device drivers Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Milestone
Release 2014.12
Development

Successfully merging this pull request may close these issues.
2 participants
@OlegHahm @thomaseichinger