Commit f79e996c authored by Nícolas F. R. A. Prado's avatar Nícolas F. R. A. Prado Committed by Daniel Lezcano

thermal/drivers/mediatek/lvts_thermal: Use offset threshold for IRQ

There are two kinds of temperature monitoring interrupts available:
* High Offset, Low Offset
* Hot, Hot to normal, Cold

The code currently uses the hot/h2n/cold interrupts, however in a way
that doesn't work: the cold threshold is left uninitialized, which
prevents the other thresholds from ever triggering, and the h2n
interrupt is used as the lower threshold, which prevents the hot
interrupt from triggering again after the thresholds are updated by the
thermal framework, since a hot interrupt can only trigger again after
the hot to normal interrupt has been triggered.

But better yet than addressing those issues, is to use the high/low
offset interrupts instead. This way only two thresholds need to be
managed, which have a simpler state machine, making them a better match
to the thermal framework's high and low thresholds.

Fixes: f5f633b1 ("thermal/drivers/mediatek: Add the Low Voltage Thermal Sensor driver")
Signed-off-by: default avatarNícolas F. R. A. Prado <nfraprado@collabora.com>
Reviewed-by: default avatarAlexandre Mergnat <amergnat@baylibre.com>
Reviewed-by: default avatarAngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Signed-off-by: default avatarDaniel Lezcano <daniel.lezcano@linaro.org>
Link: https://lore.kernel.org/r/20230706153823.201943-4-nfraprado@collabora.com
parent 64de162e
...@@ -298,9 +298,9 @@ static int lvts_set_trips(struct thermal_zone_device *tz, int low, int high) ...@@ -298,9 +298,9 @@ static int lvts_set_trips(struct thermal_zone_device *tz, int low, int high)
u32 raw_high = lvts_temp_to_raw(high); u32 raw_high = lvts_temp_to_raw(high);
/* /*
* Hot to normal temperature threshold * Low offset temperature threshold
* *
* LVTS_H2NTHRE * LVTS_OFFSETL
* *
* Bits: * Bits:
* *
...@@ -309,13 +309,13 @@ static int lvts_set_trips(struct thermal_zone_device *tz, int low, int high) ...@@ -309,13 +309,13 @@ static int lvts_set_trips(struct thermal_zone_device *tz, int low, int high)
if (low != -INT_MAX) { if (low != -INT_MAX) {
pr_debug("%s: Setting low limit temperature interrupt: %d\n", pr_debug("%s: Setting low limit temperature interrupt: %d\n",
thermal_zone_device_type(tz), low); thermal_zone_device_type(tz), low);
writel(raw_low, LVTS_H2NTHRE(base)); writel(raw_low, LVTS_OFFSETL(base));
} }
/* /*
* Hot temperature threshold * High offset temperature threshold
* *
* LVTS_HTHRE * LVTS_OFFSETH
* *
* Bits: * Bits:
* *
...@@ -323,7 +323,7 @@ static int lvts_set_trips(struct thermal_zone_device *tz, int low, int high) ...@@ -323,7 +323,7 @@ static int lvts_set_trips(struct thermal_zone_device *tz, int low, int high)
*/ */
pr_debug("%s: Setting high limit temperature interrupt: %d\n", pr_debug("%s: Setting high limit temperature interrupt: %d\n",
thermal_zone_device_type(tz), high); thermal_zone_device_type(tz), high);
writel(raw_high, LVTS_HTHRE(base)); writel(raw_high, LVTS_OFFSETH(base));
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment