[coreboot-gerrit] Change in coreboot[master]: payloads/tianocore: Enable UEFIPayload

[Matt DeVillier (Code Review)]

Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34459 )

Change subject: payloads/tianocore: Enable UEFIPayload
......................................................................

Patch Set 5:

(1 comment)

https://review.coreboot.org/c/coreboot/+/34459/5/payloads/external/tianocore... 
File payloads/external/tianocore/Kconfig:

https://review.coreboot.org/c/coreboot/+/34459/5/payloads/external/tianocore... 
PS5, Line 90: config TIANOCORE_BOOTLOADER
            : 	bool "TianoCore UEFI Payload"
            : 	help
            : 	  UEFIPayload supports coreboot and slimboot.
            :
> I still don't see why we should add an unmaintained bootloader-selection that only works with upstream. Upstream was broken quite often, that's why we switched to a stable branch.

I think the argument could be made that if it's not easy to test UefiPayloadPkg via coreboot's build system, then the likelihood of it being properly maintained drops significantly.

If we have two payload options then it makes it easier to test, and we can switch the default to UefiPayloadPkg once it reaches the point to where it's a better option