Attention is currently required from: Alan Green, Xiang Wang, Paul Menzel.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/49637 )
Change subject: ft2232_spi.c: Generalized GPIOL pin control
......................................................................
Patch Set 5: Code-Review+1
(3 comments)
Commit Message:
https://review.coreboot.org/c/flashrom/+/49637/comment/1465d95b_d9e6a9e8
PS5, Line 7: Generalized GPIOL pin control
Please make it a statement, e.g.:
Generalize GPIOL pin control
File ft2232_spi.c:
https://review.coreboot.org/c/flashrom/+/49637/comment/0c4d245f_2253b00b
PS5, Line 377: cs_bits
pinlvl ?
https://review.coreboot.org/c/flashrom/+/49637/comment/576cbe4d_a1a20494
PS5, Line 381:
Using the default value can work, but it is recommended to add the following code to make the meanin […]
This would allow overriding the settings for some programmers
--
To view, visit
https://review.coreboot.org/c/flashrom/+/49637
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I1f2b3b968577e62e3c5b11bcdf4afe2de6eb84ab
Gerrit-Change-Number: 49637
Gerrit-PatchSet: 5
Gerrit-Owner: Alan Green
avg@google.com
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Edward O'Callaghan
quasisec@chromium.org
Gerrit-Reviewer: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Reviewer: Xiang Wang
merle@hardenedlinux.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Alan Green
avg@google.com
Gerrit-Attention: Xiang Wang
merle@hardenedlinux.org
Gerrit-Attention: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Wed, 20 Jan 2021 09:27:22 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Xiang Wang
merle@hardenedlinux.org
Gerrit-MessageType: comment