summaryrefslogtreecommitdiffstats
path: root/drivers/iio
diff options
context:
space:
mode:
authorAndreas Klinger <ak@it-klinger.de>2020-05-04 20:10:34 +0200
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2020-06-25 15:41:48 +0200
commit4c852f957a1e07229cc3e1ce7e8dd2b7c11eb782 (patch)
tree24a8788a59ca1a9464168e110ade9953bfc6a77f /drivers/iio
parent2966b022f98fe102d38386f3d33deaed508a4d05 (diff)
downloadlinux-stable-4c852f957a1e07229cc3e1ce7e8dd2b7c11eb782.tar.gz
linux-stable-4c852f957a1e07229cc3e1ce7e8dd2b7c11eb782.tar.bz2
linux-stable-4c852f957a1e07229cc3e1ce7e8dd2b7c11eb782.zip
iio: bmp280: fix compensation of humidity
[ Upstream commit dee2dabc0e4115b80945fe2c91603e634f4b4686 ] Limit the output of humidity compensation to the range between 0 and 100 percent. Depending on the calibration parameters of the individual sensor it happens, that a humidity above 100 percent or below 0 percent is calculated, which don't make sense in terms of relative humidity. Add a clamp to the compensation formula as described in the datasheet of the sensor in chapter 4.2.3. Although this clamp is documented, it was never in the driver of the kernel. It depends on the circumstances (calibration parameters, temperature, humidity) if one can see a value above 100 percent without the clamp. The writer of this patch was working with this type of sensor without noting this error. So it seems to be a rare event when this bug occures. Signed-off-by: Andreas Klinger <ak@it-klinger.de> Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'drivers/iio')
-rw-r--r--drivers/iio/pressure/bmp280-core.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/iio/pressure/bmp280-core.c b/drivers/iio/pressure/bmp280-core.c
index 3204dff34e0a..ae415b4e381a 100644
--- a/drivers/iio/pressure/bmp280-core.c
+++ b/drivers/iio/pressure/bmp280-core.c
@@ -182,6 +182,8 @@ static u32 bmp280_compensate_humidity(struct bmp280_data *data,
+ (s32)2097152) * H2 + 8192) >> 14);
var -= ((((var >> 15) * (var >> 15)) >> 7) * (s32)H1) >> 4;
+ var = clamp_val(var, 0, 419430400);
+
return var >> 12;
};