[coreboot-gerrit] Change in coreboot[master]: amdfwtool: Use "*SIZE" in fw config file as NVRAM entry

[Bao Zheng (Code Review)]

Attention is currently required from: Raul Rangel, Martin Roth, Felix Held.
Bao Zheng has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/49015 )

Change subject: amdfwtool: Use "*SIZE" in fw config file as NVRAM entry
......................................................................

Patch Set 35:

(4 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/49015/comment/2f2eff82_f5a00f11 
PS1, Line 7: *
> Is "*" proper?

Done.
Keep as "*".

Commit Message:

https://review.coreboot.org/c/coreboot/+/49015/comment/cf40281d_d8ad9bb7 
PS27, Line 11: to config file. Then the required size of space will
             : be allocated related to the firmware type.
> Maybe add a comment as to why this is needed?

Done.
Added in the commit message.

Commit Message:

https://review.coreboot.org/c/coreboot/+/49015/comment/484abc63_da7fff87 
PS30, Line 7: Use *number in config file
> If this is the reason, could we add this to the commit message?

Done

https://review.coreboot.org/c/coreboot/+/49015/comment/f0ab62ca_c2ea38c2 
PS30, Line 10: XXXX_FW_TYPE_STRING   *0xXXXXX
> Why not just use commas to separate them?

Ack. The asterisk is not a separator. It is a mark to tell it is a entry without a filename.