Re: [PATCH] arm64: dts: qcom: x1e80100-dell-xps13-9345: Add thermal zones

[Abel Vesa]

On 26-10-06 14:55:57, Liviu Nicoara wrote:
> The XPS 13 9345 has seven board thermistors on the PMK8550 VADC: AMUX1
> to AMUX5 THM and AMUX1 and AMUX2 GPIO, each with a 100k pull-up. The
> ADC configuration in Dell's Windows driver package lists the same seven
> channels, SYS_THERM1 to SYS_THERM7.
> 
> The ACPI tables define seven ADC thermal zones, TZ31 to TZ37. Three
> of them, TZ32 to TZ34, carry passive trips that throttle all CPUs and
> the GPU; the other four sit at 100 C. The tables do not name a zone's
> channel. Taken in SYS_THERM order, TZ32 to TZ34 are AMUX2 GPIO, AMUX1
> THM and AMUX2 THM. Heating the board confirms two of the three: a
> charger on the left port warms AMUX2 GPIO, one on the right port warms
> AMUX2 THM. Sustained SSD reads, compared against a CPU-only load, warm
> AMUX3 THM and AMUX2 GPIO beyond the other channels; the second is the
> left charger's, so the first is named for the SSD.
> 
> The passive trips follow the thermal mode chosen in the BIOS setup
> (Thermal Management: Optimized, Cool, Quiet or Ultra Performance),
> which the EC reports and which cannot be read without an EC driver.
> Use the values of Optimized, the default: 53.8 C on the left charger,
> 62.8 C on AMUX1 THM, 61.8 C on the right charger. The critical trips
> are the tables' 114.8 C. The other four channels get report-only
> zones, as on the CRD, and all seven are put in thermal-monitor mode.
> 
> Assisted-by: LLM
> Signed-off-by: Liviu Nicoara <lnicoara@xxxxxxxxxxxx>

Tested-by: Abel Vesa <abel.vesa@xxxxxxxxxxxxxxxx>
Reviewed-by: Abel Vesa <abel.vesa@xxxxxxxxxxxxxxxx>