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

DHTSensor fixes #1979

Merged
merged 3 commits into from
Nov 9, 2019
Merged

DHTSensor fixes #1979

merged 3 commits into from
Nov 9, 2019

Conversation

mcspr
Copy link
Collaborator

@mcspr mcspr commented Nov 8, 2019 •
edited
Loading

mcspr added 2 commits November 8, 2019 20:31
@mcspr
sensor/dht: increase the delay per DHT22 datasheet
bad2855 
Operating specifications / Communication and signal

"Data-bus's free status is high voltage level. When communication
between MCU and DHT22 begin, program of MCU will transform data-bus's
voltage level from high to low level and this process must beyond at least
**1ms** to ensure DHT22 could detect MCU's signal, then MCU will wait 20-40us for
DHT22's response."
@mcspr
sensor/dht: set pin mode before digitalWrite
6861417
@mcspr
Copy link
Collaborator Author

mcspr commented Nov 9, 2019

I wonder if additional timeout when just starting up does anything...
Increased timeout should though, pinMode too.

@mcspr mcspr merged commit 08cfcbd into xoseperez:dev Nov 9, 2019
@mcspr mcspr deleted the sensor/dht-fixes branch November 9, 2019 00:28
This was referenced Nov 19, 2019
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mcspr