ron minnich has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34476 )
Change subject: Rampayload: Attempt to boot coreboot without ramstage ......................................................................
Patch Set 3:
This is much more what I was hoping was possible. But, you've introduced a new name TARGET_STAGE and then in the config.h we have ENV_PAYLOAD_LOADER
But these two things are connected in function, but not in the name. They are in some sense the terminal stage before the payload.
This seems confusing to me. Could you not have PAYLOAD_LOADER_STAGE or PRE_PAYLOAD_STAGE or FINAL_STAGE or something? Sorry, again, TARGET_STAGE doesn't communicate anything to me.
Is there some other variable name that shows a connection to the fact that the stage in question is the payload loader? Or am I missing the point here? It's just that, looking at the name TARGET_STAGE, it's hard to draw a meaning from the name. Thanks.