[coreboot-gerrit] Change in coreboot[master]: mb/google/hatch/variants/hatch: Set PCH Thermal Threshold value to 77...

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33147 )

Change subject: mb/google/hatch/variants/hatch: Set PCH Thermal Threshold value to 77 deg C
......................................................................

Patch Set 6:

(1 comment)

https://review.coreboot.org/c/coreboot/+/33147/6/src/mainboard/google/hatch/... 
File src/mainboard/google/hatch/variants/hatch/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/33147/6/src/mainboard/google/hatch/... 
PS6, Line 57: .pch_thermal_trip = 77,
> Sure, will do that.

You can add this to baseboard/devicetree.cb:

register "common_soc_config.pch_thermal_trip" = "77" and that should take effect for all variants.

One word of caution: If a variant wants to override this value, they will have to use the same format i.e register "ommon_soc_config.pch_thermal_trip" = "xy" instead of putting it under register "common_soc_config". This is because of the way overrides are currently applied in sconfig.