[coreboot-gerrit] Change in coreboot[master]: x86: Implement RESET_VECTOR_IN_DRAM option

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32414 )

Change subject: x86: Implement RESET_VECTOR_IN_DRAM option
......................................................................

Patch Set 14:

(1 comment)

https://review.coreboot.org/c/coreboot/+/32414/14//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/32414/14//COMMIT_MSG@21 
PS14, Line 21: begin executing it at reset.
Almost off-topic: If PSP shall run VBOOT, what is you plan about the toolchain used to build it? Proprietary or coreboot reference, or you throw the sources to AMD and get a signed blob back in return?

From GPL compliance guide:

Therefore, if you choose not to distribute the compiler, you should include a readme about where you got it, what version it was, and who to contact to acquire it, regardless of whether your compiler is FOSS, proprietary, or internally developed.

So, if you were to prepare coreboot toolchain anyways for verstage in PSP, I thought it would make sense to adapt rmodule loader such that PSP loads romstage as an rmodule.