On Fri, Jan 21, 2022 at 10:48:06AM +0000, Ojeda Leon, Nicolas wrote: > Hi Gerd, > > > Hmm, the QemuFwCfgHardwareInfoLib.inf file created by patch #2 has > > HardwareInfoPciHostBridgeLib.c + QemuFwCfgHardwareInfoLib.c too. > > > I'm wondering why you add two inf files in the first place? > > My idea was to provide 2 variations of the library: > > - One offering no dynamic memory and list features and only read functions > to, statically, parse data out of a fw-cfg file -> QemuFwCfgHardwareInfo > - The other one being the general one having everything. Both the list and > dynamic features as well as the fw-cfg wrappers -> HardwareInfoLib
Why do you need the former? To figure the highest address used in PEI, whereas the DXE drivers use the other one? Usual naming convention for that case would be PeiHardwareInfoLib + DxeHardwareInfoLib ... take care, Gerd -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#85919): https://edk2.groups.io/g/devel/message/85919 Mute This Topic: https://groups.io/mt/88565406/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
