Hi,

> > Are you expecting to need different functionality from the ArmVirtPkg
> > one? If not, you could move that one to
> >   OvmfPkg/Library/QemuFwCfgCommonLib
> > or something.

> The interfaces provided by ARM QemuFwLib seems works fine with RISC-V
> OVMF, however the build fail with the recent change of adding "#if
> defined(MDE_CPU_AARCH64)".

How does that fail to build?

The #ifdef looks like a performance optimization, doing 64bit mmio
access on 64bit platforms to reduce the number of traps.  Even without
fixing that up for RISCV64 so it gets the same treatment everything
should work fine, just a little slower than it could.

> I prefer to have RISC-V instance because I
> am not sure if any fw_cfg interface changes would be made on RISC-V
> QEMU.

The fw_cfg interface should not diverge.  ia32/x64 works a bit different
because there is a separate io address space on that architecture and
fw_cfg uses that for historical reasons.  All platforms using the mmio
variant of the fw_cfg interface should work alike though.

take care,
  Gerd



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#80353): https://edk2.groups.io/g/devel/message/80353
Mute This Topic: https://groups.io/mt/85405739/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to