Wim Vervoorn has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36856 )
Change subject: soc/intel/skylake/acpi/dptf: Disable DTRP when no DPTF_TSRX_SENSOR_ID is defined ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/36856/2/src/soc/intel/skylake/acpi/... File src/soc/intel/skylake/acpi/dptf/thermal.asl:
https://review.coreboot.org/c/coreboot/+/36856/2/src/soc/intel/skylake/acpi/... PS2, Line 78: #ifdef DPTF_TSR0_SENSOR_ID | DPTF_TSR1_SENSOR_ID | DPTF_TSR2_SENSOR_ID
first time I see this syntax. […]
Thanks for noticing. This was overlooked. Multiple items aren't supported by #ifdef. This is changed and now works as expected.