[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:

(2 comments)

https://review.coreboot.org/#/c/32118/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/32118/1//COMMIT_MSG@9 
PS1, Line 9: Follow thermal table for second tunning.
> Are these final tuning values based on latest system available ? OR will there be some more tuning c […]

Yes, these are tuning values based on latest devices and system.

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
> Any reason for this change ?  […]

Because Chromebook use DPTF passive policy 1, so don't need set PL2 25W to dynamic to TDP 15W, thanks.