On Wed, Apr 17, 2024 at 04:12:56PM +0800, Chao Li wrote: > Patch1: Added three PCDs for QemuFwCfgLibMmio > Patch2: Sparate QemuFwCfgLibMmio.c into two files and default as DXE > stage library. > Patch3: Added QemuFwCfgMmiLib PEI version > Patch4: Rename QemuFwCfgLibMmio.inf to QemuFwCfgMmioDxeLib.inf and > enable it in AARCH64 and RISCV64.
Ok, I see, you are using the PCDs because global variables don't work in PEI. Alternative approach would be to create a HOB for that (see EFI_HOB_PLATFORM_INFO used by X64). Not sure this is a good idea though given that we have three different architectures using that code. Ard, any advise? > OvmfPkg: Add three PCDs for QemuFwCfgLib > OvmfPkg: Separate QemuFwCfgLibMmio.c into two files This patch should be splitted into two, one doing the code split without functional change, and one which switches from global variables to PCDs (or HOB). Otherwise this looks good to me (and I'd suggest to keep and merge this as separate patch series). take care, Gerd -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#117912): https://edk2.groups.io/g/devel/message/117912 Mute This Topic: https://groups.io/mt/105573103/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
