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] TMP12x temperature sensor #757

Merged
merged 3 commits into from
Oct 21, 2021

Conversation

chris-durand
Copy link
Member

Simple driver for TMP121/TMP123/TMP125 temperature sensors. The driver has been in use for some time on a custom board with an STM32F042K6. I'll test the Nucleo example tomorrow.

@chris-durand chris-durand force-pushed the feature/tmp12x branch 2 times, most recently from b43c7bf to 62a2c00 Compare October 20, 2021 20:18
@rleh
Copy link
Member

rleh commented Oct 20, 2021

The only difference between TMP121 und TMP123 are the swapped GND and NC pins, or am I missing something?

Copy link
Member

@rleh rleh left a comment

Choose a reason for hiding this comment

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

Nice driver 👍🏽

src/modm/driver/temperature/tmp12x.hpp Outdated Show resolved Hide resolved
src/modm/driver/temperature/tmp12x.hpp Outdated Show resolved Hide resolved
@chris-durand
Copy link
Member Author

The only difference between TMP121 und TMP123 are the swapped GND and NC pins, or am I missing something?

Most likely the only difference is the chip marking. For the ones I had the NC pins were bonded to GND.

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.

Very nice!

@chris-durand
Copy link
Member Author

Please don't merge until I have tested the example.

@salkinium salkinium added this to the 2021q4 milestone Oct 20, 2021
@chris-durand
Copy link
Member Author

I have tested the example and it works.

@salkinium salkinium merged commit 624ce10 into modm-io:develop Oct 21, 2021
@chris-durand chris-durand deleted the feature/tmp12x branch October 21, 2021 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants