kernel/linux
0
Fork 0
mirror of https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/ synced 2026-10-16 22:39:48 +00:00
Code Issues Releases Packages Activity
linux/drivers/iio/humidity
History
John Brooks 12c2fcf6bc iio: dht11: Use usleep_range instead of msleep for start signal 
commit 5c113b5e00 upstream.

The DHT22 (AM2302) datasheet specifies that the LOW start pulse should not
exceed 20ms. However, observations with an oscilloscope of an RPi Model 2B
(rev 1.1) communicating with a DHT22 sensor showed that the driver was
consistently sending start pulses longer than 20ms:

Kernel 4.7.10-v7+ (n=132):
    Minimum pulse length: 20.20ms
    Maximum:              29.84ms
    Mean:                 24.96ms
    StDev:                2.82ms
    Sensor response rate: 100%
    Read success rate:    76%

On kernel 4.8, the start pulse was so long that the sensor would not even
respond 97% of the time:

Kernel 4.8.16-v7+ (n=100):
    Minimum pulse length: 30.4ms
    Maximum:              74.4ms
    Mean:                 39.3ms
    StDev:                10.2ms
    Sensor response rate: 3%
    Read success rate:    3%

The driver would return ETIMEDOUT and write log messages like this:

[   51.430987] dht11 dht11@0: Only 1 signal edges detected
[   66.311019] dht11 dht11@0: Only 0 signal edges detected

Replacing msleep(18) with usleep_range(18000, 20000) made the pulse length
sane again and restored responsiveness:

Kernel 4.8.16-v7+ with usleep_range (n=123):
    Minimum pulse length: 18.16ms
    Maximum:              20.20ms
    Mean:                 19.85ms
    StDev:                0.51ms
    Sensor response rate: 100%
    Read success rate:    84%

Signed-off-by: John Brooks <john@fastquake.com>
Reviewed-by: Harald Geyer <harald@ccbib.org>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
History 2017-02-09 08:08:30 +01:00
..
am2315.c iio: humidity: am2315: set up buffer timestamps for non-zero values 2016-08-15 15:39:23 +01:00
dht11.c iio: dht11: Use usleep_range instead of msleep for start signal 2017-02-09 08:08:30 +01:00
hdc100x.c iio: humidity: hdc100x: fix sensor data reads of temp and humidity 2016-08-15 17:01:53 +01:00
htu21.c iio: Export I2C module alias information 2016-05-21 20:08:14 +01:00
Kconfig Merge 4.8-rc5 into staging-next 2016-09-05 08:13:24 +02:00
Makefile iio: humidity: Add support for AM2315 2016-04-24 09:56:07 +01:00
si7005.c iio: convert to common i2c_check_functionality() return value 2016-02-27 17:17:43 +00:00
si7020.c iio: convert to common i2c_check_functionality() return value 2016-02-27 17:17:43 +00:00