summaryrefslogtreecommitdiffstats
path: root/drivers/hwmon/sparx5-temp.c
diff options
context:
space:
mode:
authorLars Povlsen <lars.povlsen@microchip.com>2020-09-03 15:47:04 +0200
committerGuenter Roeck <linux@roeck-us.net>2020-09-23 18:39:40 +0200
commitfcb575bfac80d6deb4940d2ebf506d6701761f38 (patch)
tree5efe7f1c4dc6913b3a05abe3c5d45a0760fceeeb /drivers/hwmon/sparx5-temp.c
parentLinux 5.9-rc6 (diff)
downloadlinux-fcb575bfac80d6deb4940d2ebf506d6701761f38.tar.xz
linux-fcb575bfac80d6deb4940d2ebf506d6701761f38.zip
hwmon: (sparx5) Fix initial reading of temperature
If the temperature is read before the internal calibration is completed, the driver returns -EIO. Instead it should return -EAGAIN to encourage repeating the operation. Note (groeck): Returning -EAGAIN to userspace may result in hard loops; some userspace code interprets -EAGAIN as request to retry immediately. I would prefer -ENODATA, but it turns out that the thermal subsystem only handles -EAGAIN silently, so we'll have to stick with that. Signed-off-by: Lars Povlsen <lars.povlsen@microchip.com> Link: https://lore.kernel.org/r/20200903134704.8949-1-lars.povlsen@microchip.com Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers/hwmon/sparx5-temp.c')
-rw-r--r--drivers/hwmon/sparx5-temp.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hwmon/sparx5-temp.c b/drivers/hwmon/sparx5-temp.c
index 1a2b1026b026..98be48e3a22a 100644
--- a/drivers/hwmon/sparx5-temp.c
+++ b/drivers/hwmon/sparx5-temp.c
@@ -56,7 +56,7 @@ static int s5_read(struct device *dev, enum hwmon_sensor_types type,
case hwmon_temp_input:
stat = readl_relaxed(hwmon->base + TEMP_STAT);
if (!(stat & TEMP_STAT_VALID))
- return -EIO;
+ return -EAGAIN;
value = stat & TEMP_STAT_TEMP;
/*
* From register documentation: