build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49015 )
Change subject: amdfwtool: Use *number in config file as NVRAM entry ......................................................................
Patch Set 2:
(3 comments)
https://review.coreboot.org/c/coreboot/+/49015/2/util/amdfwtool/amdfwtool.c File util/amdfwtool/amdfwtool.c:
https://review.coreboot.org/c/coreboot/+/49015/2/util/amdfwtool/amdfwtool.c@... PS2, Line 1035: } else if (fw_table[i].blank_size != 0){ space required before the open brace '{'
https://review.coreboot.org/c/coreboot/+/49015/2/util/amdfwtool/data_parse.c File util/amdfwtool/data_parse.c:
https://review.coreboot.org/c/coreboot/+/49015/2/util/amdfwtool/data_parse.c... PS2, Line 258: psp_tableptr->blank_size = strtoull(&filename[1], NULL, 16); line over 96 characters
https://review.coreboot.org/c/coreboot/+/49015/2/util/amdfwtool/data_parse.c... PS2, Line 337: bhd_tableptr->blank_size = strtoull(&filename[1], NULL, 16); line over 96 characters