Attention is currently required from: Ravi kumar, Paul Menzel, mturney mturney.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56093 )
Change subject: mb/google/herobrine: Initialize USB by calling SOC method
......................................................................
Patch Set 34:
(2 comments)
File src/mainboard/google/herobrine/mainboard.c:
https://review.coreboot.org/c/coreboot/+/56093/comment/a88b390e_441e6490
PS34, Line 19: void mainboard_blob_fix(void);
What is this and what does it have to do with this patch?
https://review.coreboot.org/c/coreboot/+/56093/comment/ad57c000_cd6c95fe
PS34, Line 23: usb0_board_data
If the SNPS doesn't use any board_data, just pass NULL (and don't define a struct for it that is never used).
--
To view, visit
https://review.coreboot.org/c/coreboot/+/56093
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic378352a97e4f3ed89089f1f7545f8ebb172b1f2
Gerrit-Change-Number: 56093
Gerrit-PatchSet: 34
Gerrit-Owner: Ravi kumar
rbokka@codeaurora.org
Gerrit-Reviewer: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Sandeep Maheswaram
sanm@codeaurora.org
Gerrit-Reviewer: Shelley Chen
shchen@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Reviewer: mturney mturney
mturney@codeaurora.org
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-CC: Ravi Kumar Bokka
c_rbokka@qualcomm.corp-partner.google.com
Gerrit-Attention: Ravi kumar
rbokka@codeaurora.org
Gerrit-Attention: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: mturney mturney
mturney@codeaurora.org
Gerrit-Comment-Date: Mon, 08 Nov 2021 17:50:44 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment