Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38755 )
Change subject: nb/amd/{agesa,pi}/acpi: include thermal zone ......................................................................
Patch Set 1:
Patch Set 1: Code-Review+1
I can confirm the register compatibility. But... reading thermal_mixin.asl, that code doesn't seem to make much sense. The ThermalZone is only reported as present if HTC (hardware temperature control) is enable, so far so good. But then the limit that would trigger HTC is used as _CRT (immediate shut- down) and (-10K) as _HOT (better power down now). Unless the HTC limit register is configured very wrong, this would force the OS to shut down rather early.
On apu2 it worked well, but I agree that an incorrectly programmed limit can cause trouble. I will look for the bits that enable HTC and program the limit (probably in AGESA) if they exist. If not will add a preceding patch that ensures the correct programming of these bits.