Attention is currently required from: Name of user not set #1004065, Stefan Reinauer. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/60736 )
Change subject: util/ifdtool: Add additional regions for platforms that support them. ......................................................................
Patch Set 1:
(4 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/60736/comment/79b17f71_c05ab810 PS1, Line 7: util/ifdtool: Add additional regions for platforms that support them. Please remove the dot/period at the end of the commit message summary.
https://review.coreboot.org/c/coreboot/+/60736/comment/4a4ae165_19cff032 PS1, Line 9: Some Intel SoCs … like X …
https://review.coreboot.org/c/coreboot/+/60736/comment/1ada13b1_1c7b2cde PS1, Line 9: Some Intel SoCs support additional IFD regions for things like embedded LAN FW or : Innovation Engine firmware, etc. Please reflow for 72 characters per line.
https://review.coreboot.org/c/coreboot/+/60736/comment/018f1a60_b7e31c9d PS1, Line 11: Maybe add TEST= with an example command, you tested.