Re: [PATCH] thermal/drivers/mediatek/lvts_thermal: Return error in case of invalid efuse data

From: AngeloGioacchino Del Regno
Date: Wed Jun 05 2024 - 06:34:10 EST


Il 04/06/24 18:33, Nicolas Pitre ha scritto:
On Tue, 4 Jun 2024, AngeloGioacchino Del Regno wrote:

Il 03/06/24 14:06, Julien Panis ha scritto:
This patch prevents from registering thermal entries and letting the
driver misbehave if efuse data is invalid. A device is not properly
calibrated if the golden temperature is zero.

Signed-off-by: Julien Panis <jpanis@xxxxxxxxxxxx>

Fixes tag, please.

What should it refer to? This is arguably a robustness enhancement for a
defficiency that has been there all along.



If that's been there since day 1, the fixes tag should be the day 1 commit,
of course.

I see that as an important bugfix and not just a robustness enhancement, as
SoCs with uncalibrated LVTS would *not work*, as the kernel would shut down
the system before even loading the userspace.

Regards,
Angelo