Attention is currently required from: Edward O'Callaghan.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/63601 )
Change subject: raiden_debug_spi.c: Allow custom_rst param value of 'false'
......................................................................
Patch Set 2: Code-Review+2
(1 comment)
File raiden_debug_spi.c:
https://review.coreboot.org/c/flashrom/+/63601/comment/86e77834_6e05e438
PS2, Line 1417: else if (!strcasecmp(custom_rst_str, "false"))
nit: All branches should use braces.
--
To view, visit
https://review.coreboot.org/c/flashrom/+/63601
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I25bfe6f8e3f7cfffb1a9c99ac90ec56a750d7f84
Gerrit-Change-Number: 63601
Gerrit-PatchSet: 2
Gerrit-Owner: Edward O'Callaghan
quasisec@chromium.org
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Attention: Edward O'Callaghan
quasisec@chromium.org
Gerrit-Comment-Date: Tue, 12 Apr 2022 23:49:04 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment