Attention is currently required from: Ravi kumar, Paul Menzel, Ravi Kumar Bokka.
Shelley Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/55950 )
Change subject: herobrine: get boardid from GPIO configuration
......................................................................
Patch Set 5:
(1 comment)
File src/mainboard/google/herobrine/boardid.c:
https://review.coreboot.org/c/coreboot/+/55950/comment/1a08116a_fb3f4bfb
PS5, Line 25: CB_ERR
hi shelly, […]
Yes, you are correct actually. But, I just realized that if we don't match one of the boards, it is actually not an error, but we just don't recognize it. So, I believe that the correct action is to return the retrieved boardid anyway and perhaps print out that it was not recognized.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/55950
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I2b7625f9b98563438d1ac20e6f29411ef1058cf4
Gerrit-Change-Number: 55950
Gerrit-PatchSet: 5
Gerrit-Owner: Ravi kumar
rbokka@codeaurora.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-CC: Ravi Kumar Bokka
c_rbokka@qualcomm.corp-partner.google.com
Gerrit-CC: Shelley Chen
shchen@google.com
Gerrit-Attention: Ravi kumar
rbokka@codeaurora.org
Gerrit-Attention: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: Ravi Kumar Bokka
c_rbokka@qualcomm.corp-partner.google.com
Gerrit-Comment-Date: Tue, 13 Jul 2021 19:06:46 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Shelley Chen
shchen@google.com
Comment-In-Reply-To: Ravi Kumar Bokka
c_rbokka@qualcomm.corp-partner.google.com
Gerrit-MessageType: comment