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.

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

[driver] Add driver for STTS22H temperature sensor #544

Merged
merged 2 commits into from
May 8, 2021

Conversation

chris-durand
Copy link
Member

@chris-durand chris-durand commented Jan 26, 2021

Simple driver for STTS22H temperature sensor. It has been successfully tested on a custom board. I will also test the example on real hardware but not with the actual Nucleo board since the chip has a 2x2mm UDFN package.

TODO:

  • Test example in hardware

@chris-durand chris-durand changed the title Add driver for STTS22H temperature sensor [driver] Add driver for STTS22H temperature sensor Jan 26, 2021
Copy link
Member

@salkinium salkinium left a comment

Choose a reason for hiding this comment

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

Please squash the docs commit into the driver commit.

examples/nucleo_f103rb/stts22h/main.cpp Outdated Show resolved Hide resolved
src/modm/driver/temperature/stts22h.hpp Show resolved Hide resolved
@chris-durand
Copy link
Member Author

Please squash the docs commit into the driver commit.

Forgot to do it, now done.

Copy link
Member

@salkinium salkinium left a comment

Choose a reason for hiding this comment

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

Any update on this? Driver looks perfectly fine to me.

@chris-durand
Copy link
Member Author

The driver is working fine and tested on a custom board with an F107. I can't test the example since I don't have any spare sensors to hook up to a Nucleo board and it would also be painful with that tiny UDFN package. From my side good to merge.

@chris-durand chris-durand added the ci:hal Triggers the exhaustive HAL compile CI jobs label May 8, 2021
@salkinium salkinium merged commit 9e7ec34 into modm-io:develop May 8, 2021
@chris-durand chris-durand deleted the feature/stts22h branch May 8, 2021 20:40
@salkinium salkinium added this to the 2021q2 milestone Jun 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:hal Triggers the exhaustive HAL compile CI jobs example 🔑 feature 🚧
Development

Successfully merging this pull request may close these issues.

2 participants