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

fxas21002 gyroscope reading is in deg/s #30446

Closed
aloebs29 opened this issue Dec 4, 2020 · 0 comments · Fixed by #30506
Closed

fxas21002 gyroscope reading is in deg/s #30446

aloebs29 opened this issue Dec 4, 2020 · 0 comments · Fixed by #30506
Assignees
@MaureenHelm
Labels
area: Sensors Sensors bug The issue is a bug, or the PR is fixing a bug

Comments

@aloebs29
Copy link

aloebs29 commented Dec 4, 2020

Describe the bug
The fxas21002 gyroscope driver scales it's output to deg/s rather than rad/s (which is what the docs say SENSOR_CHANNEL_GYRO_* should output).

To Reproduce
Run the fxas21002 sample project and observe the output. This can also be observed analytically by comparing the fxas21002_convert() function in fxas21002.c to the datasheet values for deg/s per LSB.

Expected behavior
Output data is in rad/s.

Impact
Minimal
@aloebs29 aloebs29 added the bug The issue is a bug, or the PR is fixing a bug label Dec 4, 2020
@carlescufi carlescufi added the area: Sensors Sensors label Dec 6, 2020
@MaureenHelm MaureenHelm mentioned this issue Dec 7, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MaureenHelm MaureenHelm

Labels
area: Sensors Sensors bug The issue is a bug, or the PR is fixing a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

drivers: sensor: Fix fxas21002 gyro units
3 participants
@MaureenHelm @carlescufi @aloebs29