Matt DeVillier has submitted this change. ( https://review.coreboot.org/c/coreboot/+/81894?usp=email )
Change subject: payloads/edk2/Makefile: Drop duplicated build string option ......................................................................
payloads/edk2/Makefile: Drop duplicated build string option
The `PRIORITIZE_INTERNAL` option was somehow duplicated, so remove the extra copy, leaving the one under the MrChromebox repo specific settings.
TEST=build qemu w/edk2 payload, check build log that the 'PRIORITIZE_INTERNAL' option is only added once to the build string.
Change-Id: I4c4c433184d93337c926e256e77054afc00a2566 Signed-off-by: Matt DeVillier matt.devillier@gmail.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/81894 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Sean Rhodes sean@starlabs.systems --- M payloads/external/edk2/Makefile 1 file changed, 0 insertions(+), 4 deletions(-)
Approvals: build bot (Jenkins): Verified Sean Rhodes: Looks good to me, approved
diff --git a/payloads/external/edk2/Makefile b/payloads/external/edk2/Makefile index dd95942..3b73ce8 100644 --- a/payloads/external/edk2/Makefile +++ b/payloads/external/edk2/Makefile @@ -79,10 +79,6 @@ ifneq ($(CONFIG_ECAM_MMCONF_LENGTH),) BUILD_STR += --pcd gEfiMdePkgTokenSpaceGuid.PcdPciExpressBaseSize=$(CONFIG_ECAM_MMCONF_LENGTH) endif -# PRIORITIZE_INTERNAL = FALSE -ifeq ($(CONFIG_EDK2_PRIORITIZE_INTERNAL),y) -BUILD_STR += -D PRIORITIZE_INTERNAL=TRUE -endif # PS2_KEYBOARD_ENABLE = FALSE ifeq ($(CONFIG_EDK2_PS2_SUPPORT),y) BUILD_STR += -D PS2_KEYBOARD_ENABLE=TRUE