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

[John Su (Code Review)]

John Su has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32118 )

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

Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/32118/1/src/mainboard/google/sarien/variants... 
File src/mainboard/google/sarien/variants/sarien/devicetree.cb:

https://review.coreboot.org/#/c/32118/1/src/mainboard/google/sarien/variants... 
PS1, Line 45: 15
> Did you check above mentioned BUG:b:122343940 ? You only requested some time back to change it from  […]

For issue:122343940 PL1 set 25W is need to leverage Window system's setting at first, but Windows system use passive policy 2.0, so issue:122343940 modify PL1 to 25W is not really a issue, it just want to leverage Window system's setting, and Window check performance by benchmark, but Chromebook by Aquarium, both are different.

Due to Chromebook performance check spec is aquarium FPS:60, so PL1 change to 15W not impact performance also FPS 60,too, thanks.