Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37263 )
Change subject: mb/google: Add unaligned flag (%) sections that don't need alignment ......................................................................
Patch Set 14:
(4 comments)
https://review.coreboot.org/c/coreboot/+/37263/5/src/mainboard/google/auron/... File src/mainboard/google/auron/chromeos.fmd:
https://review.coreboot.org/c/coreboot/+/37263/5/src/mainboard/google/auron/... PS5, Line 7: RW_SECTION_A@0x0 0xf0000 {
The point is that it is never safe to have a section unaligned when […]
Done
https://review.coreboot.org/c/coreboot/+/37263/5/src/mainboard/google/auron/... PS5, Line 20: SHARED_DATA@0x0 0x2000
Why do we need these to be aligned? […]
Done
https://review.coreboot.org/c/coreboot/+/37263/5/src/mainboard/google/auron/... PS5, Line 23: RW_VPD(PRESERVE)@0x1f8000 0x2000
RW_VPD is handled by flashrom so we don't need it
Done
https://review.coreboot.org/c/coreboot/+/37263/5/src/mainboard/google/auron/... PS5, Line 26: RW_LEGACY(CBFS)@0x240000 0x1c0000
RW_LEGACY is updated by flashrom so we don't need it
Done