[coreboot-gerrit] Change in coreboot[master]: mb/google/zork: update berknip CHTC thermal setting

[Chris Wang (Code Review)]

Chris Wang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47158 )

Change subject: mb/google/zork: update berknip CHTC thermal setting
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/47158/2/src/mainboard/google/zork/v... 
File src/mainboard/google/zork/variants/berknip/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/47158/2/src/mainboard/google/zork/v... 
PS2, Line 9: 	register "system_config" = "3"
> This still looks wrong to me. […]

Yes, That should set "2" for TDP 15w and "3" for 25w. and since it also set the sustained_power_limit to 12w so it will be limited. Checking with IRM, the difference between "2" and "3" is only the sustained_power_limit. but suggest setting the system_config to "2" to 
prevent potential risk.