Attention is currently required from: Bhanu Prakash Maiya, Rob Barnes, Eric Peers, Karthik Ramasubramanian.
Kevin Chiu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59051 )
Change subject: mb/google/guybrush: Update SPKR GPIO configuration for guybrush/nipperkin
......................................................................
Patch Set 5:
(2 comments)
File src/mainboard/google/guybrush/variants/baseboard/gpio.c:
https://review.coreboot.org/c/coreboot/+/59051/comment/d378add4_18c2c402
PS4, Line 70: PAD_NC(GPIO_31),
Guybrush reference board v2 still uses GPIO_31 for EN_SPKR. So this needs to be overridden there.
Hi Rob,
per the suggestion from Karthik, I'll add GPIO_31 override in guybrush variant, thanks.
https://review.coreboot.org/c/coreboot/+/59051/comment/f0995530_cc686f8e
PS4, Line 87: PAD_NC(GPIO_70),
Please configure the following GPIOs in the baseboard here. […]
Done
--
To view, visit
https://review.coreboot.org/c/coreboot/+/59051
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I3d82292b116f53d85d9518364ffd2169bd915a7e
Gerrit-Change-Number: 59051
Gerrit-PatchSet: 5
Gerrit-Owner: Kevin Chiu
kevin.chiu.17802@gmail.com
Gerrit-Reviewer: Bhanu Prakash Maiya
bhanumaiya@google.com
Gerrit-Reviewer: Eric Peers
epeers@google.com
Gerrit-Reviewer: Karthik Ramasubramanian
kramasub@google.com
Gerrit-Reviewer: Rob Barnes
robbarnes@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Bhanu Prakash Maiya
bhanumaiya@google.com
Gerrit-Attention: Rob Barnes
robbarnes@google.com
Gerrit-Attention: Eric Peers
epeers@google.com
Gerrit-Attention: Karthik Ramasubramanian
kramasub@google.com
Gerrit-Comment-Date: Tue, 16 Nov 2021 02:30:08 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Rob Barnes
robbarnes@google.com
Comment-In-Reply-To: Karthik Ramasubramanian
kramasub@google.com
Gerrit-MessageType: comment