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

[Lance Zhao (Code Review)]

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

Change subject: [WIP]payloads/tianocore: Enable UEFIPayload
......................................................................

Patch Set 1:
> Patch Set 1:
> > Patch Set 1:
> >
> > I've not had a chance to boot-test this patch yet, but it is insufficient as-is with a default config due to the default being my repo/my coreboot_fb branch. One would need to change the git revision to upstream/master for this to build even.
> >
> > Until UefiPayloadPkg reaches feature/functional parity with my existing fork, what might make sense is to add a Kconfig option to select CorebootPayloadPkg (using my repo/branch) or UefiPayloadPkg (using upstream/master) and leave the default as-is
>
> I tested a separately build UefiPayloadPkg (qemu). Specifying the PCIE_BASE was needed and PS2 keyboard did not work. Other than that, it does boot.

Okay, PCIE_BASE is needed. Did you get a chance to try on real platform, last time I tried and that can't boot up as well. May I know what you have done to add PCIE_BASE? Thanks.