diff options
| author | Hans de Goede <hdegoede@redhat.com> | 2015-03-21 15:02:55 +0100 |
|---|---|---|
| committer | Eduardo Valentin <edubezval@gmail.com> | 2015-04-07 13:11:29 -0700 |
| commit | 7e497a7375fe5d5c5d751a71d006f4fd515bbcd8 (patch) | |
| tree | 9c521532a0c0647fe29e10cad1f6bc2abdc5b8d2 /drivers/thermal | |
| parent | 2771d00081a4e70f52a9b290dae1b68b1bdf41f6 (diff) | |
| download | linux-7e497a7375fe5d5c5d751a71d006f4fd515bbcd8.tar.gz linux-7e497a7375fe5d5c5d751a71d006f4fd515bbcd8.tar.bz2 linux-7e497a7375fe5d5c5d751a71d006f4fd515bbcd8.zip | |
thermal: Do not log an error if thermal_zone_get_temp returns -EAGAIN
Some temperature sensors only get updated every few seconds and while
waiting for the first irq reporting a (new) temperature to happen there
get_temp operand will return -EAGAIN as it does not have any data to report
yet.
Not logging an error in this case avoids messages like these from showing
up in dmesg on affected systems:
[ 1.219353] thermal thermal_zone0: failed to read out thermal zone 0
[ 2.015433] thermal thermal_zone0: failed to read out thermal zone 0
[ 2.416737] thermal thermal_zone0: failed to read out thermal zone 0
Reviewed-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
Diffstat (limited to 'drivers/thermal')
| -rw-r--r-- | drivers/thermal/thermal_core.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c index 174d3bcf8bd7..4108db7e10c1 100644 --- a/drivers/thermal/thermal_core.c +++ b/drivers/thermal/thermal_core.c @@ -458,8 +458,10 @@ static void update_temperature(struct thermal_zone_device *tz) ret = thermal_zone_get_temp(tz, &temp); if (ret) { - dev_warn(&tz->device, "failed to read out thermal zone %d\n", - tz->id); + if (ret != -EAGAIN) + dev_warn(&tz->device, + "failed to read out thermal zone (%d)\n", + ret); return; } |