Raul Rangel has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/54134 )
Change subject: mb/google/guybrush: Add SoC thermal zone ......................................................................
mb/google/guybrush: Add SoC thermal zone
The time constant values were taken from the zork thermal.asl.
BUG=b:186166365 TEST=Boot guybrush to OS and verify logs look correct thermal-0294 thermal_trips_update : Found critical threshold [3641] thermal-0321 thermal_trips_update : No hot threshold thermal-0200 thermal_get_temperatur: Temperature is 3060 dK thermal-0219 thermal_get_polling_fr: Polling frequency is 100 dS thermal-0200 thermal_get_temperatur: Temperature is 3060 dK thermal LNXTHERM:00: registered as thermal_zone0 ACPI: Thermal Zone [TM00] (33 C) thermal-0200 thermal_get_temperatur: Temperature is 3070 dK
Signed-off-by: Raul E Rangel rrangel@chromium.org Change-Id: Iaeed75bdaa16b117d0fa7144ede98db1388f74f3 --- M src/mainboard/google/guybrush/Kconfig M src/mainboard/google/guybrush/variants/baseboard/devicetree.cb 2 files changed, 25 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/34/54134/1
diff --git a/src/mainboard/google/guybrush/Kconfig b/src/mainboard/google/guybrush/Kconfig index 8c23cf0..9dba79c 100644 --- a/src/mainboard/google/guybrush/Kconfig +++ b/src/mainboard/google/guybrush/Kconfig @@ -11,6 +11,7 @@ select BOARD_ROMSIZE_KB_16384 select DISABLE_KEYBOARD_RESET_PIN select DISABLE_SPI_FLASH_ROM_SHARING + select DRIVERS_ACPI_THERMAL_ZONE select DRIVERS_AMD_I2S_MACHINE_DEV select DRIVERS_GENERIC_GPIO_KEYS select DRIVERS_I2C_GENERIC diff --git a/src/mainboard/google/guybrush/variants/baseboard/devicetree.cb b/src/mainboard/google/guybrush/variants/baseboard/devicetree.cb index 1d01fff..1a019b4 100644 --- a/src/mainboard/google/guybrush/variants/baseboard/devicetree.cb +++ b/src/mainboard/google/guybrush/variants/baseboard/devicetree.cb @@ -52,6 +52,30 @@ register "i2c_scl_reset" = "GPIO_I2C0_SCL | GPIO_I2C1_SCL | GPIO_I2C2_SCL | GPIO_I2C3_SCL"
+ chip drivers/acpi/thermal_zone + register "name" = ""SOC"" + + register "temperature_sensor_id" = "0" + + register "polling_period" = "10000" + + # EC is configured to power off the system at 92C, so add one degree of buffer + # so the OS can gracefully shutdown + register "critical_temperature" = "91" + + # EC is configured to assert PROCHOT at 90C. That drastically lowers + # performance. Instead we will tell the OS to start throttling the CPUs at + # 85C in hopes that we don't hit the PROCHOT limit. + register "passive_config" = "{ + .temperature = 85, + .time_constant_1 = 2, + .time_constant_2 = 5, + .time_sampling_period = 2000, + }" + + device generic 0 on end + end + device domain 0 on device ref gpp_bridge_0 on end # WLAN device ref gpp_bridge_1 on end # SD