Attention is currently required from: Kevin Chiu, Subrata Banik, Tim Wawrzynczak, Paul Menzel, Angel Pons, Nick Vaccaro, Name of user not set #1004174, Shon Wang.
Robert Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/60331 )
Change subject: mb/google/brya/var/vell: Change AMP driver setting
......................................................................
Patch Set 33:
(3 comments)
File src/mainboard/google/brya/variants/vell/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/60331/comment/bf2b58f8_af3beb92
PS32, Line 173: device ref i2c0 on
these are all missing the entry to set them to external boost: […]
Done
https://review.coreboot.org/c/coreboot/+/60331/comment/170f0cf6_8b12d34a
PS32, Line 177: register "boost_peak_milliamp" = "4500"
: register "boost_ind_nanohenry" = "BOOST_IND_1000_NH"
: register "boost_cap_microfarad" = "24"
these aren't required for external boost. […]
Done
https://review.coreboot.org/c/coreboot/+/60331/comment/d6a2dcc1_ebecff95
PS32, Line 182: register "gpio_src_select[0]" = "GPIO_SRC_HIGH_IMPEDANCE"
the driver will override this anyway, so not critical, but gpio_src_select[0] should be GPIO_SRC_GPI […]
Done
--
To view, visit
https://review.coreboot.org/c/coreboot/+/60331
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I96d49bd1a2ba061c4fd52b450b31d0885f49552c
Gerrit-Change-Number: 60331
Gerrit-PatchSet: 33
Gerrit-Owner: Shon Wang
shon.wang@quanta.corp-partner.google.com
Gerrit-Reviewer: Kevin Chiu
kevin.chiu.17802@gmail.com
Gerrit-Reviewer: Name of user not set #1004174
Gerrit-Reviewer: Nick Vaccaro
nvaccaro@google.com
Gerrit-Reviewer: Shon Wang
shon.wang@quanta.corp-partner.google.com
Gerrit-Reviewer: Subrata Banik
subratabanik@google.com
Gerrit-Reviewer: Tim Wawrzynczak
twawrzynczak@chromium.org
Gerrit-Reviewer: YH Lin
yueherngl@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Angel Pons
th3fanbus@gmail.com
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-CC: Robert Chen
robert.chen@quanta.corp-partner.google.com
Gerrit-Attention: Kevin Chiu
kevin.chiu.17802@gmail.com
Gerrit-Attention: Subrata Banik
subratabanik@google.com
Gerrit-Attention: Tim Wawrzynczak
twawrzynczak@chromium.org
Gerrit-Attention: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: Angel Pons
th3fanbus@gmail.com
Gerrit-Attention: Nick Vaccaro
nvaccaro@google.com
Gerrit-Attention: Name of user not set #1004174
Gerrit-Attention: Shon Wang
shon.wang@quanta.corp-partner.google.com
Gerrit-Comment-Date: Tue, 01 Mar 2022 03:19:23 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Name of user not set #1004174
Gerrit-MessageType: comment