Il 02/04/24 05:25, Nicolas Pitre ha scritto:
This is a bunch of patches to support the MT8186 and MT8188 thermal sensor configurations. Several changes are needed to cope with oddities these SOCs implement. All values (calibration data offsets, etc.) were lifted and adapted from the vendor driver source code.
I picked patches 7 and 12 (and also fixed them) introducing the nodes for the LVTS controllers, but will not pick 9 and 15, as they're either missing thermal zones and/or using the wrong names; let's wait for the next cycle for those, as I will also be able to add the SVS on top (needs a bit of time for testing), getting both SoCs complete on the LVTS side, without rushing. Cheers, Angelo
Changes from v2: - use meaningful name for binding index definitions - reuse LVTS_COEFF_*_MT7988 on MT8186 per reviewer request - do similarly for MT8188 that now reuses LVTS_COEFF_*_MT8195 - use thermal zone names the svs driver wants - adjust some DT node names and iospace length - remove variable .hw_tshut_temp as it is constant across all SOCs Version 2 can be found here: https://lore.kernel.org/all/20240318212428.3843952-1-nico@xxxxxxxxxxx/ Changes from v1: - renamed CPU cluster thermal zones in DT - fixed logic to cope with empty controller slots at the beginning - isolated bindings to their own patches - added MT8188 default thermal zones Version 1 can be found here: https://lore.kernel.org/all/20240111223020.3593558-1-nico@xxxxxxxxxxx/T/ diffstat: .../thermal/mediatek,lvts-thermal.yaml | 6 + arch/arm64/boot/dts/mediatek/mt8186.dtsi | 256 +++++++++++ arch/arm64/boot/dts/mediatek/mt8188.dtsi | 383 ++++++++++++++++ drivers/thermal/mediatek/lvts_thermal.c | 434 +++++++++++++----- .../thermal/mediatek,lvts-thermal.h | 26 ++ 5 files changed, 987 insertions(+), 118 deletions(-)