[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/+/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
> Because Chromebook use DPTF passive policy 1, so don't need set PL2 25W to dynamic to TDP 15W, thank […]

Did you check above mentioned BUG:b:122343940 ? You only requested some time back to change it from 15W to 25W. And now you are changing it back to 15W on this bug c#1.

Also, did you get chance to check the performance benchmark impact due to this your new change from 25W to 15W ?