[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/acpi/dptf: Add fan based active cooling for TSR sensors

[Tim Wawrzynczak (Code Review)]

Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35127 )

Change subject: soc/intel/common/acpi/dptf: Add fan based active cooling for TSR sensors
......................................................................

Patch Set 5: -Code-Review

(1 comment)

Good point Fur

https://review.coreboot.org/c/coreboot/+/35127/5/src/soc/intel/common/acpi/d... 
File src/soc/intel/common/acpi/dptf/thermal.asl:

PS5:
> Just curious: Why are these controls added only to TSR0 and TSR1 but not to TSR2 and TSR3?

That's a good point; I think for Hatch we were only using TSR0/1 but if this is in common code, it should be available for 2 and 23 as well.