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

[Michał Żygowski (Code Review)]

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:
>
> Sorry, there's some misunderstanding. I didn't doubt that the
> HTC feature is configured correctly.
>
> I doubt that the thermal_mix.asl code is actually useful with
> correctly configured HTC. If I read it correctly, it would
> prevent HTC to become active (powers down long before that
> would happen).

Okay, so are you referring to this line?
Return (Subtract (_CRT, K10TEMP_HOT_OFFSET))

Or I am still missing your point (or something obvious)?