[coreboot-gerrit] Change in coreboot[master]: nb/amd/{agesa,pi}/acpi: include thermal zone

[Nico Huber (Code Review)]

Nico Huber 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: 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.