12345678910111213141516171819202122232425262728293031323334353637383940 |
- From 27b389d9f62c2174f95fe4002b11e77d4cb3ce80 Mon Sep 17 00:00:00 2001
- From: =?UTF-8?q?N=C3=ADcolas=20F=2E=20R=2E=20A=2E=20Prado?=
- <[email protected]>
- Date: Thu, 6 Jul 2023 11:37:32 -0400
- Subject: [PATCH 25/42] thermal/drivers/mediatek/lvts_thermal: Handle IRQ on
- all controllers
- MIME-Version: 1.0
- Content-Type: text/plain; charset=UTF-8
- Content-Transfer-Encoding: 8bit
- There is a single IRQ handler for each LVTS thermal domain, and it is
- supposed to check each of its underlying controllers for the origin of
- the interrupt and clear its status. However due to a typo, only the
- first controller was ever being handled, which resulted in the interrupt
- never being cleared when it happened on the other controllers. Add the
- missing index so interrupts are handled for all controllers.
- Fixes: f5f633b18234 ("thermal/drivers/mediatek: Add the Low Voltage Thermal Sensor driver")
- Reviewed-by: Matthias Brugger <[email protected]>
- Reviewed-by: AngeloGioacchino Del Regno <[email protected]>
- Tested-by: Chen-Yu Tsai <[email protected]>
- Signed-off-by: Nícolas F. R. A. Prado <[email protected]>
- Reviewed-by: Alexandre Mergnat <[email protected]>
- Signed-off-by: Daniel Lezcano <[email protected]>
- Link: https://lore.kernel.org/r/[email protected]
- ---
- drivers/thermal/mediatek/lvts_thermal.c | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
- --- a/drivers/thermal/mediatek/lvts_thermal.c
- +++ b/drivers/thermal/mediatek/lvts_thermal.c
- @@ -451,7 +451,7 @@ static irqreturn_t lvts_irq_handler(int
-
- for (i = 0; i < lvts_td->num_lvts_ctrl; i++) {
-
- - aux = lvts_ctrl_irq_handler(lvts_td->lvts_ctrl);
- + aux = lvts_ctrl_irq_handler(&lvts_td->lvts_ctrl[i]);
- if (aux != IRQ_HANDLED)
- continue;
-
|