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

Now that you mention it one option forward could be to add 
QemuFwCfgHardwareInfo as a dependency in HardwareInfoLib.
Or we could have a single inf file (the general one) and not use the dynamic 
functions where not possible, if you think this would be a cleaner solution.

Thank you for your comment.

Best regards,

Nicolas



Amazon Development Center Germany GmbH
Krausenstr. 38
10117 Berlin
Geschaeftsfuehrung: Christian Schlaeger, Jonathan Weiss
Eingetragen am Amtsgericht Charlottenburg unter HRB 149173 B
Sitz: Berlin
Ust-ID: DE 289 237 879





-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#85924): https://edk2.groups.io/g/devel/message/85924
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]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to