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

[Bao Zheng (Code Review)]

Bao Zheng 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 49:

(4 comments)

https://review.coreboot.org/c/coreboot/+/42859/47//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/42859/47//COMMIT_MSG@9 
PS47, Line 9: Internal Cleanup:
> shouldn't this be just "b:"?

Done. Changed.

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

PS47:
> since now all platforms are updated in this patch, this copy of the old amdfwtool can be dropped fro […]

Done. Removed.

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

https://review.coreboot.org/c/coreboot/+/42859/47/util/amdfwtool/amdfwtool.h... 
PS47, Line 117: endif
> #endif /* _AMD_FW_TOOL_H_ */

Done

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");
> Good job, build bot.

Done.Fix typo.