[coreboot-gerrit] Change in ...coreboot[master]: util/amdfwtool: Clarify calculations with structures

[Martin Roth (Code Review)]

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

Change subject: util/amdfwtool: Clarify calculations with structures
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/#/c/31727/2/util/amdfwtool/amdfwtool.c 
File util/amdfwtool/amdfwtool.c:

https://review.coreboot.org/#/c/31727/2/util/amdfwtool/amdfwtool.c@809 
PS2, Line 809: sizeof(embedded_firmware);
> Since this is 24 bytes (0x18), i'm not sure this is right if we were expecting the following table t […]

Ok, and I see where this is updated in a later change with an alignment macro as well.