On Thu, Sep 03, 2020 at 03:47:04PM +0200, Lars Povlsen wrote: > 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. > > Signed-off-by: Lars Povlsen <lars.povl...@microchip.com>
Applied. I added a note stating that I would prefer -ENODATA, and why, but that this is not feasible due to thermal subsystem requirements. Thanks, Guenter > --- > drivers/hwmon/sparx5-temp.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > -- > 2.27.0 > > 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:
