Attention is currently required from: Chen Wisley, Sumeet R Pawnikar, Wisley Chen.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58970 )
Change subject: mb/google/brya/var/redrix: Add two thermal sensor setting
......................................................................
Patch Set 1:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/58970/comment/6d703a12_b7eadcff
PS1, Line 7: mb/google/brya/var/redrix: Add two thermal sensor setting
1. Add thermal sensors for SOC and voltage regulator
or
2. Hook up two missing sensors
File src/mainboard/google/brya/variants/redrix/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/58970/comment/37ed3fd6_c9cb7488
PS1, Line 88: register "options.tsr[2].desc" = ""Charger""
So, the charger sensor was incorrect before?
--
To view, visit
https://review.coreboot.org/c/coreboot/+/58970
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ia9c58129d439ade21e96896c5e593cd08a627603
Gerrit-Change-Number: 58970
Gerrit-PatchSet: 1
Gerrit-Owner: Chen Wisley
wisley.chen@quantatw.com
Gerrit-Reviewer: Sumeet R Pawnikar
sumeet.r.pawnikar@intel.com
Gerrit-Reviewer: Tim Wawrzynczak
twawrzynczak@chromium.org
Gerrit-Reviewer: Wisley Chen
wisley.chen@quanta.corp-partner.google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: Chen Wisley
wisley.chen@quantatw.com
Gerrit-Attention: Sumeet R Pawnikar
sumeet.r.pawnikar@intel.com
Gerrit-Attention: Wisley Chen
wisley.chen@quanta.corp-partner.google.com
Gerrit-Comment-Date: Fri, 05 Nov 2021 13:59:38 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment