drivers: add driver for DS18B20 temperature sensors - #14899
Closed
haukepetersen wants to merge 14 commits into
Closed
drivers: add driver for DS18B20 temperature sensors#14899haukepetersen wants to merge 14 commits into
haukepetersen wants to merge 14 commits into
Conversation
haukepetersen
added
Type: new feature
haukepetersen
requested review from
MichelRottleuthner,
aabadie,
fjmolinas,
leandrolanzieri and
smlng
as code owners
August 31, 2020 09:31
Contributor
Author
|
pushed some minor doxygen and test app type fixes |
haukepetersen
force-pushed
the
add_driver_ds18b20
branch
from
August 31, 2020 09:37
34ed8de to
0ce0a55
Compare
haukepetersen
removed
the
CI: ready for build
haukepetersen
force-pushed
the
add_driver_ds18b20
branch
from
September 3, 2020 11:59
0ce0a55 to
66266a4
Compare
Contributor
Author
haukepetersen
force-pushed
the
add_driver_ds18b20
branch
from
September 7, 2020 08:46
66266a4 to
2ddfb58
Compare
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you want me to ignore this issue, please mark it with the "State: don't stale" label. Thank you for your contributions. |
stale
Bot
added
the
State: stale
haukepetersen
removed
the
State: stale
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you want me to ignore this issue, please mark it with the "State: don't stale" label. Thank you for your contributions. |
stale
Bot
added
the
State: stale
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contribution description
This PR adds a new device driver for
ds18b20temperature sensors. It is build on top of the generic 1-Wire bus driver (PRed in #14897) and supports any number of sensors connected to the same bus.I PRed it separately of the existing
ds18driver, as it enables users to chose whichever version suits their application best...Testing procedure
Connect any number of
ds18b20sensors to a board of your choice, flash the included test application, and use theinit,read, andread_seqshell commands to sample the connected sensors.Issues/PRs references
rebased on #14897