Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32414 )
Change subject: arch/x86: Add option for running romstage in DRAM ......................................................................
Patch Set 5:
Hey Julius, thanks for your thoughts. Besides what Martin said, I have a number of other items on my mind. Some were hinted at in the documentation and have not made their way into the patch stack yet. But we're trying to get this system up in a short amount of time, minimize the number of unforeseen pitfalls, and with minimal disruption to coreboot's x86 methodology. We've started capturing ideas for ways to streamline the bootflow once the system is fully functional, and ultimately I'd hope to pull this particular change back out once it's no longer useful.
All the other stuff you're doing (e.g. switching to protected mode) is custom to your platform regardless (romstage normally starts in protected mode already). Inserting that at the start of ramstage shouldn't be any harder than inserting it at the start of romstage?
Right, creating a hybrid ramstage would be no harder than a hybrid romstage, I think. Either way, it's a custom solution for Family 17h though.