[coreboot-gerrit] [M] Change in coreboot[master]: amdfwtool: Allow the location to be a relative address

[Martin L Roth (Code Review)]

Attention is currently required from: Bao Zheng, ritul guru, Zheng Bao.

Martin L Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69856 )

Change subject: amdfwtool: Allow the location to be a relative address
......................................................................

Patch Set 17:

(2 comments)

File util/amdfwtool/amdfwtool.c:

https://review.coreboot.org/c/coreboot/+/69856/comment/c20b8df2_12850d0c 
PS17, Line 2499: /* Relative address. */
What does this comment mean?  Is it for the things above the comment? If it's for things below the comment, maybe move it after the break?

https://review.coreboot.org/c/coreboot/+/69856/comment/76208804_d1ad7d3a 
PS17, Line 2505: /* Special cases for 8M */
What if a ROM smaller than 8MiB is used on an older platform?  Would we need more special cases?