Attention is currently required from: Hung-Te Lin, Rex-BC Chen.
Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58838 )
Change subject: mb/google/corsola: Add NOR-Flash support
......................................................................
Patch Set 3:
(1 comment)
File src/mainboard/google/corsola/bootblock.c:
PS3:
some of the gpio struct definitions and handling code in here looks to me that it should probably be moved to the soc code, since setting up the spi gpio pads won't be the only place where gpio pads need to be set up. if you prefer to do that at a later point in time, feel free to ack this one
--
To view, visit
https://review.coreboot.org/c/coreboot/+/58838
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iee3dd336632b0cf998f5f7c1d118e01e8270e815
Gerrit-Change-Number: 58838
Gerrit-PatchSet: 3
Gerrit-Owner: Rex-BC Chen
rex-bc.chen@mediatek.corp-partner.google.com
Gerrit-Reviewer: Hung-Te Lin
hungte@chromium.org
Gerrit-Reviewer: Rex-BC Chen
rex-bc.chen@mediatek.corp-partner.google.com
Gerrit-Reviewer: Yu-Ping Wu
yupingso@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Felix Held
felix-coreboot@felixheld.de
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-Attention: Hung-Te Lin
hungte@chromium.org
Gerrit-Attention: Rex-BC Chen
rex-bc.chen@mediatek.corp-partner.google.com
Gerrit-Comment-Date: Thu, 04 Nov 2021 02:07:28 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment