Attention is currently required from: Wisley Chen. Hello Wisley Chen,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/58970
to review the following change.
Change subject: mb/google/brya/var/redrix: Add two thermal sensor setting ......................................................................
mb/google/brya/var/redrix: Add two thermal sensor setting
Redrix has 4 thermal sensors, so add the missing sensors settings.
BUG=b:200134784 TEST=build and verified by thermal team.
Signed-off-by: Wisley Chen wisley.chen@quanta.corp-partner.google.com Change-Id: Ia9c58129d439ade21e96896c5e593cd08a627603 --- M src/mainboard/google/brya/variants/redrix/overridetree.cb 1 file changed, 7 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/70/58970/1
diff --git a/src/mainboard/google/brya/variants/redrix/overridetree.cb b/src/mainboard/google/brya/variants/redrix/overridetree.cb index 0ed6c52..370957f 100644 --- a/src/mainboard/google/brya/variants/redrix/overridetree.cb +++ b/src/mainboard/google/brya/variants/redrix/overridetree.cb @@ -84,13 +84,18 @@ chip drivers/intel/dptf ## sensor information register "options.tsr[0].desc" = ""DRAM"" - register "options.tsr[1].desc" = ""Charger"" + register "options.tsr[1].desc" = ""SOC"" + register "options.tsr[2].desc" = ""Charger"" + register "options.tsr[3].desc" = ""5V regulator"" + # TODO: below values are initial reference values only ## Passive Policy register "policies.passive" = "{ [0] = DPTF_PASSIVE(CPU, CPU, 55, 5000), [1] = DPTF_PASSIVE(CPU, TEMP_SENSOR_0, 45, 5000), - [2] = DPTF_PASSIVE(CHARGER, TEMP_SENSOR_1, 51, 5000), + [2] = DPTF_PASSIVE(CPU, TEMP_SENSOR_1, 51, 5000), + [3] = DPTF_PASSIVE(CHARGER, TEMP_SENSOR_2, 51, 5000), + [4] = DPTF_PASSIVE(CPU, TEMP_SENSOR_3, 51, 5000), }" register "controls.power_limits" = "{ .pl1 = {