Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32725 )
Change subject: Replace ENV_RAMSTAGE with ENV_PAYLOAD_LOADER ......................................................................
Patch Set 4:
(7 comments)
It looks like you have replaced all occurrences of ENV_RAMSTAGE with ENV_PAYLOAD_LOADER.
[Subrata] Move majority of stuffs because in future in anyone wish to call those functionally as well for an example GDB. but to enable CONFIG_RAMPAYLOAD we will have dedicated CL to just include required files in postcar stage.
But, is the intent to really enable all the different blocks for postcar when it is the payload loader? e.g. pci drivers? microcode loader? others? How is the ramstage different than postcar in that case?
[Subrata] No, we won't include those in Makefile.inc while selecting CONFIG_RAMPAYLOAD sample CL: https://review.coreboot.org/#/c/coreboot/+/32675/