Attention is currently required from: Arthur Heymans.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81777?usp=email )
The change is no longer submittable: All-Comments-Resolved is unsatisfied now.
Change subject: Makefile.mk: Account for large code model sections in cbfs_struct ......................................................................
Patch Set 2:
(1 comment)
File Makefile.mk:
https://review.coreboot.org/c/coreboot/+/81777/comment/7cba8b68_ec83580a : PS2, Line 369: $(OBJCOPY_ramstage) -O binary --only-section='.data*' --only-section='.bss*' \ Would probably be simpler to just say `--only-section='.*data*' --only-section='.*bss*'` here. In fact this really should be picking up `.rodata` anyway, we're just getting lucky that nobody is marking their structures in these files as `const`. The section restriction was just added originally to exclude `.note.gnu...` stuff.