On 2016-05-09 01:57:30, Laszlo Ersek wrote:
> On 05/08/16 06:31, Abner Chang wrote:
> > From: AbnerChang <[email protected]>
> > 
> > Use OVMF PCI host bridge driver as the RISC-V platform BUS. This driver is
> > used by RISC-V Virtualization package (RiscVVirtPkg).
> > Currently the platfrom spec for RISC-V is not yet ready, thus we use PCI 
> > host
> > bridge in temporarily.
> > 
> > Contributed-under: TianoCore Contribution Agreement 1.0
> > Signed-off-by: Abner Chang<[email protected]>
> > ---
> >  OvmfPkg/PciHostBridgeDxe/Ia32/IoFifo.S        |  134 ++
> >  OvmfPkg/PciHostBridgeDxe/Ia32/IoFifo.asm      |  140 ++
> >  OvmfPkg/PciHostBridgeDxe/IoFifo.h             |  176 ++
> >  OvmfPkg/PciHostBridgeDxe/PciHostBridge.c      | 1551 +++++++++++++++
> >  OvmfPkg/PciHostBridgeDxe/PciHostBridge.h      |  651 ++++++
> >  OvmfPkg/PciHostBridgeDxe/PciHostBridgeDxe.inf |   66 +
> >  OvmfPkg/PciHostBridgeDxe/PciRootBridgeIo.c    | 2628 
> > +++++++++++++++++++++++++
> >  OvmfPkg/PciHostBridgeDxe/X64/IoFifo.S         |  122 ++
> >  OvmfPkg/PciHostBridgeDxe/X64/IoFifo.asm       |  126 ++
> >  9 files changed, 5594 insertions(+)
> >  create mode 100644 OvmfPkg/PciHostBridgeDxe/Ia32/IoFifo.S
> >  create mode 100644 OvmfPkg/PciHostBridgeDxe/Ia32/IoFifo.asm
> >  create mode 100644 OvmfPkg/PciHostBridgeDxe/IoFifo.h
> >  create mode 100644 OvmfPkg/PciHostBridgeDxe/PciHostBridge.c
> >  create mode 100644 OvmfPkg/PciHostBridgeDxe/PciHostBridge.h
> >  create mode 100644 OvmfPkg/PciHostBridgeDxe/PciHostBridgeDxe.inf
> >  create mode 100644 OvmfPkg/PciHostBridgeDxe/PciRootBridgeIo.c
> >  create mode 100644 OvmfPkg/PciHostBridgeDxe/X64/IoFifo.S
> >  create mode 100644 OvmfPkg/PciHostBridgeDxe/X64/IoFifo.asm
> 
> Nacked-by: Laszlo Ersek <[email protected]>
> 
> It took forever to get rid of this driver in OvmfPkg. RISC-V appears to
> be a brand new platform for edk2. Please work with Ray so that the core
> host bridge driver can be used with RISC-V.
> 

I agree with this. Why can't we work towards using the common driver
as we have for OVMF IA32/X64. Bringing this back (even under a
RiscV*Pkg) seems like taking a short cut ..

-Jordan
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to