Nick Vaccaro has submitted this change. ( https://review.coreboot.org/c/coreboot/+/56819 )
Change subject: mb/google/hatch/scout: Update DPTF parameters ......................................................................
mb/google/hatch/scout: Update DPTF parameters
update the DPTF parameters received from the thermal team.
BUG=b:195602767 TEST=emerge-ambassador coreboot
Signed-off-by: Kenneth Chan kenneth.chan@quanta.corp-partner.google.com Change-Id: I5dc89d1d4c2b64c9aac780a7db743a91fd0ebc9b Reviewed-on: https://review.coreboot.org/c/coreboot/+/56819 Reviewed-by: Tim Wawrzynczak twawrzynczak@chromium.org Reviewed-by: Jeff Chase jnchase@google.com Reviewed-by: Sumeet R Pawnikar sumeet.r.pawnikar@intel.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/mainboard/google/hatch/variants/scout/overridetree.cb 1 file changed, 10 insertions(+), 8 deletions(-)
Approvals: build bot (Jenkins): Verified Sumeet R Pawnikar: Looks good to me, approved Tim Wawrzynczak: Looks good to me, approved Jeff Chase: Looks good to me, but someone else must approve Kenneth Chan: Looks good to me, but someone else must approve
diff --git a/src/mainboard/google/hatch/variants/scout/overridetree.cb b/src/mainboard/google/hatch/variants/scout/overridetree.cb index a6e1150..8b7d558 100644 --- a/src/mainboard/google/hatch/variants/scout/overridetree.cb +++ b/src/mainboard/google/hatch/variants/scout/overridetree.cb @@ -240,20 +240,22 @@ register "policies.active[0]" = "{.target=DPTF_CPU, .thresholds={TEMP_PCT(94, 0),}}" register "policies.active[1]" = "{.target=DPTF_TEMP_SENSOR_0, - .thresholds={TEMP_PCT(72, 90), - TEMP_PCT(68, 80), - TEMP_PCT(62, 70), - TEMP_PCT(54, 60), - TEMP_PCT(46, 50), - TEMP_PCT(39, 40),}}" + .thresholds={TEMP_PCT(84, 90), + TEMP_PCT(82, 80), + TEMP_PCT(80, 70), + TEMP_PCT(66, 60), + TEMP_PCT(52, 50), + TEMP_PCT(35, 40),}}"
## Passive Policy register "policies.passive[0]" = "DPTF_PASSIVE(CPU, CPU, 95, 5000)" - register "policies.passive[1]" = "DPTF_PASSIVE(CPU, TEMP_SENSOR_0, 75, 5000)" + register "policies.passive[1]" = "DPTF_PASSIVE(CPU, TEMP_SENSOR_0, 85, 5000)" + register "policies.passive[2]" = "DPTF_PASSIVE(CPU, TEMP_SENSOR_1, 85, 5000)"
## Critical Policy register "policies.critical[0]" = "DPTF_CRITICAL(CPU, 100, SHUTDOWN)" - register "policies.critical[1]" = "DPTF_CRITICAL(TEMP_SENSOR_0, 85, SHUTDOWN)" + register "policies.critical[1]" = "DPTF_CRITICAL(TEMP_SENSOR_0, 90, SHUTDOWN)" + register "policies.critical[2]" = "DPTF_CRITICAL(TEMP_SENSOR_1, 90, SHUTDOWN)"
## Power Limits Control # PL1 is fixed at 15W, avg over 28-32s interval