Paul Menzel 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 1:
(5 comments)
As this is quite a big design change for coreboot, maybe it should be brought up on the mailing list.
https://review.coreboot.org/#/c/32414/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/32414/1//COMMIT_MSG@7 PS1, Line 7: Romstage romstage
https://review.coreboot.org/#/c/32414/1//COMMIT_MSG@10 PS1, Line 10: Romstage romstage
https://review.coreboot.org/#/c/32414/1/src/Kconfig File src/Kconfig:
https://review.coreboot.org/#/c/32414/1/src/Kconfig@153 PS1, Line 153: x86 You spell x86 differently X86 in the commit messages, I believe.
https://review.coreboot.org/#/c/32414/1/src/Kconfig@154 PS1, Line 154: falling is falling? falls?
https://review.coreboot.org/#/c/32414/1/src/arch/x86/assembly_entry.S File src/arch/x86/assembly_entry.S:
https://review.coreboot.org/#/c/32414/1/src/arch/x86/assembly_entry.S@29 PS1, Line 29: IS_ENABLED CONFIG