ron minnich has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32709 )
Change subject: Define ENV_PAYLOAD_LOADER
......................................................................
Patch Set 2:
(1 comment)
OK, take a look.
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
Or just move ENV_PAYLOAD_LOADER outside of these conditionals and set it to ENV_RAMSTAGE. […]
Aaron, I thought about that, I like that as well. Let me redo this and you can see what you think.
subrata, next step is to replace ENV_RAMSTAGE *where appropriate*, yes.
--
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: 2
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 03:54:13 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Aaron Durbin
adurbin@chromium.org
Comment-In-Reply-To: Subrata Banik
subrata.banik@intel.com
Gerrit-MessageType: comment