Attention is currently required from: Chen Wisley, Paul Menzel.
Wisley Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58970 )
Change subject: mb/google/brya/var/redrix: Hook up two missing sensors
......................................................................
Patch Set 2:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/58970/comment/09aa6d09_a384d18b
PS1, Line 7: mb/google/brya/var/redrix: Add two thermal sensor setting
- Add thermal sensors for SOC and voltage regulator […]
Done
File src/mainboard/google/brya/variants/redrix/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/58970/comment/1ceba6d7_6ed52eaf
PS1, Line 88: register "options.tsr[2].desc" = ""Charger""
So, the charger sensor was incorrect before?
yes.
--
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: 2
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: Paul Menzel
paulepanter@mailbox.org
Gerrit-Comment-Date: Mon, 08 Nov 2021 07:51:53 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel
paulepanter@mailbox.org
Gerrit-MessageType: comment