[coreboot-gerrit] Change in ...coreboot[master]: mb/google/sarien/variants/sarien: 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/+/31170 )

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

Patch Set 5:

(2 comments)

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

https://review.coreboot.org/#/c/31170/5/src/mainboard/google/sarien/variants... 
PS5, Line 42: 5000
I am not clear on this. Please, explain why this is such a high number.

https://review.coreboot.org/#/c/31170/5/src/mainboard/google/sarien/variants... 
PS5, Line 67: 3000
How did you arrive to this number ? I would suggest to keep this at least same as PL1 minimum i.e. 25W.