Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32709 )
Change subject: Define ENV_PAYLOAD_LOADER
......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/32709/1/src/include/rules.h
File src/include/rules.h:
https://review.coreboot.org/#/c/32709/1/src/include/rules.h@92
PS1, Line 92: 1
now our next step would be replace all ENV_RAMSTAGE with ENV_PAYLOAD_LOADER ?
Or just move ENV_PAYLOAD_LOADER outside of these conditionals and set it to ENV_RAMSTAGE. Then, when adding more logic, update the definition?
--
To view, visit
https://review.coreboot.org/c/coreboot/+/32709
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I7d4aa71bad92987374d57ff350b9b0178ee7c12b
Gerrit-Change-Number: 32709
Gerrit-PatchSet: 1
Gerrit-Owner: ron minnich
rminnich@gmail.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: Aaron Durbin
adurbin@chromium.org
Gerrit-Comment-Date: Fri, 10 May 2019 02:34:12 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Subrata Banik
subrata.banik@intel.com
Gerrit-MessageType: comment