On 17/08/2023 10:45, Chen-Yu Tsai wrote:
On Thu, Aug 17, 2023 at 4:49 AM Daniel Lezcano
<daniel.lezcano@xxxxxxxxxx> wrote:
Hi,
On 16/08/2023 21:57, Nícolas F. R. A. Prado wrote:
[ ... ]
Hi Daniel,
just a gentle reminder. As you've just applied [1], there are no longer any
concerns with this series, and it'll provide both working interrupts and
reliable thermal readings on MT8192.
There are still concerns and questions in the series for patch2 and 3.
FWIW the readout errors raised in patch 3 were fixed by
"thermal/drivers/mediatek/lvts_thermal: Make readings valid in filtered mode"
So I guess the remaining concern is on patch 2 about whether the noirq
suspend callback should be used instead.
Yes, and the reordering of the lvts_of_match
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog