sensor: mpu6050: switch to devicetree and provide sample #21565
Conversation
pabigot
requested review from
MaureenHelm,
dbkinder and
galak
as code owners
Add a binding for the sensor and replace all Kconfig selection of hardware parameters with devicetree property values. Signed-off-by: Peter Bigot <[email protected]>
pabigot
force-pushed
the
nordic/20191222a
branch
from
8b39d58 to
332b595
Compare
samples/sensor/mpu6050/README.rst
Outdated
There was a problem hiding this comment.
Use either samples or measures, but not both words. Since you say "samples" below, I'd change this then to say just samples too.
There was a problem hiding this comment.
I went with measures/measurements to avoid confusion with the fact this is a sample application.
pabigot
force-pushed
the
nordic/20191222a
branch
from
332b595 to
d31673f
Compare
zephyrbot
added
the
area: Tests
|
All checks are passing now. Tip: The bot edits this comment instead of posting a new one, so you can check the comment's history to see earlier messages. |
samples/sensor/mpu6050/src/main.c
Outdated
Add a sample for the MPU6050 that demonstrates on-demand and triggered display of all sensor data. Signed-off-by: Peter Bigot <[email protected]>
pabigot
force-pushed
the
nordic/20191222a
branch
from
d31673f to
0cf89fa
Compare
MPU6050 no longer using Kconfig to specify I2C addresses. Reference to the removed symbol causes QA diagnostics. Remove the reference; when AK8975 is converted to devicetree it may be possible to restore the link between the sensors. Signed-off-by: Peter Bigot <[email protected]>
|
@MaureenHelm Units fixed. I added a patch to the ak8975 internal header to eliminate the Kconfig warning, but that device needs to be converted to devicetree. |
MaureenHelm
left a comment
MaureenHelm
left a comment
There was a problem hiding this comment.
Units fixed. I added a patch to the ak8975 internal header to eliminate the Kconfig warning, but that device needs to be converted to devicetree.
Thanks.
| reg = <0x68>; | ||
| status = "okay"; | ||
| label = "MPU6050"; | ||
| int-gpios = <&gpio0 11 0>; |
There was a problem hiding this comment.
Actually not missing here: this is targeting master, and GPIO_ACTIVE_HIGH isn't available yet.
| reg = <0x68>; | ||
| status = "okay"; | ||
| label = "MPU6050"; | ||
| int-gpios = <&gpio0 11 0>; |
6 participants
Converts the driver to use devicetree properties instead of Kconfig.
Adds a sample demonstrating on-demand and triggered sampling.
NB: I find that the driver returns
-EIOon sample fetch within the first hundred observations. It seems likely some timing constraint is being violated.