[coreboot-gerrit] Change in ...coreboot[master]: mb/google/sarien/variants/arcada: Update thermal configuration for DPTF

[Sumeet R Pawnikar (Code Review)]

Sumeet R Pawnikar has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31331 )

Change subject: mb/google/sarien/variants/arcada: Update thermal configuration for DPTF
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/#/c/31331/2/src/mainboard/google/sarien/variants... 
File src/mainboard/google/sarien/variants/arcada/include/variant/acpi/dptf.asl:

https://review.coreboot.org/#/c/31331/2/src/mainboard/google/sarien/variants... 
PS2, Line 23: 108
> The purpose of TSR0 and TSR2 critical would be preventing from serious issue like casing melting. […]

If intention here to prevent any high temperature serious issue, in that case, you should set this critical value to lower value <100. I not clear on keeping this such a high value, how will you prevent serious issue as you mentioned above.