Hi Angelo,

Gesendet: Montag, 27. Oktober 2025 um 11:45
Von: "AngeloGioacchino Del Regno" [off-list ref]
Betreff: Re: [PATCH v1 4/5] thermal/drivers/mediatek/lvts_thermal: Add SoC based golden Temp

Il 26/10/25 13:21, Frank Wunderlich ha scritto:

quoted

From: Frank Wunderlich <redacted>

Add SoC based golden temp for invalid efuse data.

This is a preliminary patch for mt7987 support where goldentemp is
slightly higher than other SOCs.

I've found this "hack" required for all of the preproduction SoCs, as those are
usually unfused and/or missing at least some calibration parameters.

Are you using an early/preproduction/whatever SoC, or are you testing on a retail
board?

thanks for looking into it. I'm using a Bananapi R4 Lite v1.1 so not pre-production SoC afaik.

Not sure why i ported this patch from sdk (maybe due to patch compatibility and thinking this is needed).
Tested without it and it seems to work as far as i can tell. Will drop it in next round.

For the no-irq part i also ported this from SDK where no irq is defined (i also wondered about missing irq-support as it would require polling to handle emergency shutdown which seems not done in sdk too):

https://git01.mediatek.com/plugins/gitiles/openwrt/feeds/mtk-openwrt-feeds/+/HEAD/autobuild/unified/filogic/24.10/files/target/linux/mediatek/files-6.6/arch/arm64/boot/dts/mediatek/mt7987.dtsi#556

https://git01.mediatek.com/plugins/gitiles/openwrt/feeds/mtk-openwrt-feeds/+/HEAD/autobuild/unified/filogic/24.10/files/target/linux/mediatek/patches-6.6/999-2101-thermal-mediatek-add-mt7987-lvts-support.patch#160

the irq_enable part is squashed with the mt7988 lvts support:

https://git01.mediatek.com/plugins/gitiles/openwrt/feeds/mtk-openwrt-feeds/+/HEAD/autobuild/unified/filogic/24.10/files/target/linux/mediatek/patches-6.6/999-2100-thermal-mediatek-add-mt7988-lvts-support.patch#110

regards Frank

Regards,
Angelo

quoted

Signed-off-by: Frank Wunderlich <redacted>
---
  drivers/thermal/mediatek/lvts_thermal.c | 13 ++++++++++++-
  1 file changed, 12 insertions(+), 1 deletion(-)