> -----Original Message----- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > Laszlo Ersek > Sent: Thursday, June 13, 2019 10:42 PM > To: devel@edk2.groups.io; Wu, Hao A > Cc: Ni, Ray; Justen, Jordan L; Ard Biesheuvel > Subject: Re: [edk2-devel] [PATCH v1 2/6] OvmfPkg/OvmfPkg.dec: Add > PcdShellFile in OVMF DEC file > > Hi Hao, > > On 05/27/19 07:19, Wu, Hao A wrote: > > REF:https://bugzilla.tianocore.org/show_bug.cgi?id=1843 > > > > OVMF is currently using the PCD 'PcdShellFile' defined in > > IntelFrameworkModulePkg. > > > > This commit will add this PCD into the OVMF DEC file in order to drop > > OVMF's dependency on IntelFrameworkModulePkg. The PCD token value > 0x17 is > > selected to fill the hole (seems the only one left) in OvmfPkg. > > > > Please note that, instead of adding the PCD under section: > > [PcdsFixedAtBuild, PcdsPatchableInModule] > > > > as in IntelFrameworkModulePkg.dec file, it is added in section: > > [PcdsFixedAtBuild] > > > > in OvmfPkg.dec instead. > > Please replace patches #2 through #4 with the following approach: > > - eliminate PcdShellFile completely, from OvmfPkg > > - in PlatformBootManagerLib, please consume gUefiShellFileGuid, from > ShellPkg.dec. > > (This is what ArmPkg and ArmVirtPkg do.)
Got it. A new series has been sent out to address this. Replaced patch #2~#4 with the last patch of the new series: https://edk2.groups.io/g/devel/message/42415 Best Regards, Hao Wu > > Thanks! > Laszlo > > > Cc: Ray Ni <ray...@intel.com> > > Cc: Jordan Justen <jordan.l.jus...@intel.com> > > Cc: Laszlo Ersek <ler...@redhat.com> > > Cc: Ard Biesheuvel <ard.biesheu...@linaro.org> > > Signed-off-by: Hao A Wu <hao.a...@intel.com> > > --- > > OvmfPkg/OvmfPkg.dec | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/OvmfPkg/OvmfPkg.dec b/OvmfPkg/OvmfPkg.dec > > index 9640360f62..38a2d4763d 100644 > > --- a/OvmfPkg/OvmfPkg.dec > > +++ b/OvmfPkg/OvmfPkg.dec > > @@ -160,6 +160,9 @@ [PcdsFixedAtBuild] > > # For the corresponding bits, 0 = Edge triggered and 1 = Level > > triggered. > > > gUefiOvmfPkgTokenSpaceGuid.Pcd8259LegacyModeEdgeLevel|0x0000|UIN > T16|0x5 > > > > + ## FFS filename to find the shell application. > > + gUefiOvmfPkgTokenSpaceGuid.PcdShellFile|{ 0x83, 0xA5, 0x04, 0x7C, > 0x3E, 0x9E, 0x1C, 0x4F, 0xAD, 0x65, 0xE0, 0x52, 0x68, 0xD0, 0xB4, > 0xD1 }|VOID*|0x17 > > + > > ## Indicates if BiosVideo driver will switch to 80x25 Text VGA Mode when > > # exiting boot service. > > # TRUE - Switch to Text VGA Mode. > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#42416): https://edk2.groups.io/g/devel/message/42416 Mute This Topic: https://groups.io/mt/31806545/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-