Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32414 )
Change subject: arch/x86: Add option for running romstage out of RAM ......................................................................
Patch Set 2:
(2 comments)
Here's the current general flow - I'm working on a design doc and will publish something both internally and at coreboot.org about the boot flow.
Is this doc ready? That would be helpful to understand the changes that are being made here.
https://review.coreboot.org/#/c/32414/2/src/arch/x86/assembly_entry.S File src/arch/x86/assembly_entry.S:
https://review.coreboot.org/#/c/32414/2/src/arch/x86/assembly_entry.S@19 PS2, Line 19: #if CONFIG(C_ENVIRONMENT_BOOTBLOCK) So,these boards are supposed to select C_ENVIRONMENT_BOOTBLOCK even if they won't have a true bootblock?
https://review.coreboot.org/#/c/32414/2/src/arch/x86/assembly_entry.S@49 PS2, Line 49: CAR_GLOBAL If the ROMSTAGE starts in RAM, why is CAR_GLOBAL area required?