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.

View Change

To view, visit change 34476. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I22994c11317cf6936828c07fcac2cf14fca8a74b
Gerrit-Change-Number: 34476
Gerrit-PatchSet: 3
Gerrit-Owner: Subrata Banik <subrata.banik@intel.com>
Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org>
Gerrit-Reviewer: Julius Werner <jwerner@chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Subrata Banik <subrata.banik@intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Reviewer: ron minnich <rminnich@gmail.com>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Comment-Date: Tue, 23 Jul 2019 17:07:22 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment