Change in coreboot[master]: amdfwtool: Make sure the level is an legal value - coreboot-gerrit

1 Mar 2022


      Attention is currently required from: Zheng Bao.
build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62487 )
Change subject: amdfwtool: Make sure the level is an legal value
......................................................................
Patch Set 1:
(1 comment)
File util/amdfwtool/data_parse.c:
Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-142816):
https://review.coreboot.org/c/coreboot/+/62487/comment/9c2d1dcf_2262cc43 
PS1, Line 537: 				assert (ch_lvl == 'x' || ch_lvl == 'X' ||
space prohibited between function name and open parenthesis '('
-- 
To view, visit https://review.coreboot.org/c/coreboot/+/62487
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie058d2ad48acbf9c3f219c472632cddceddfb825
Gerrit-Change-Number: 62487
Gerrit-PatchSet: 1
Gerrit-Owner: Bao Zheng fishbaozi@gmail.com
Gerrit-Reviewer: Zheng Bao
Gerrit-CC: Paul Menzel paulepanter@mailbox.org
Gerrit-CC: build bot (Jenkins) no-reply@coreboot.org
Gerrit-Attention: Zheng Bao
Gerrit-Comment-Date: Tue, 01 Mar 2022 11:35:42 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment