[coreboot-gerrit] Change in coreboot[master]: amdfwtool: Take a config file instead of command line parameters

[build bot (Jenkins) (Code Review)]

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42859 )

Change subject: amdfwtool: Take a config file instead of command line parameters
......................................................................

Patch Set 47:

(3 comments)

https://review.coreboot.org/c/coreboot/+/42859/47/util/amdfwtool/amdfwtool-b... 
File util/amdfwtool/amdfwtool-b.c:

https://review.coreboot.org/c/coreboot/+/42859/47/util/amdfwtool/amdfwtool-b... 
PS47, Line 221: 	printf("                               Stoneyridge, Raven, Picasso, Renoir or Lucienne");
line over 96 characters

https://review.coreboot.org/c/coreboot/+/42859/47/util/amdfwtool/amdfwtool.c 
File util/amdfwtool/amdfwtool.c:

https://review.coreboot.org/c/coreboot/+/42859/47/util/amdfwtool/amdfwtool.c... 
PS47, Line 169: 	printf("-p | --load-mp2-fw             Set if load MP2 firmare\n");
'firmare' may be misspelled - perhaps 'firmware'?

https://review.coreboot.org/c/coreboot/+/42859/47/util/amdfwtool/amdfwtool.c... 
PS47, Line 219: 	printf("-d | --debug                   \n");
unnecessary whitespace before a quoted newline