Re: [PATCH] RISC-V: defconfig: Enable Generic PCIE by default

From: Alistair Francis
Date: Fri Dec 21 2018 - 14:22:47 EST


On Fri, 2018-12-21 at 11:13 -0800, Paul Walmsley wrote:
> Hello Alistair,
>
> On Fri, 21 Dec 2018, Alistair Francis wrote:
>
> > Enable generic PCIe by default in the RISC-V defconfig, this allows
> > us
> > to use QEMU's PCIe support out of the box. Also remove the Xilinx
> > PCIe
> > support by default as this is rarely used and conflicts with the
> > more
> > commonly used (out of tree) MicroSemi PCIe root complex.
>
> There are many folks working on FPGAs who use the Xilinx driver.
>
> Also, we should not remove support for an in-tree driver because an
> out-of-tree driver conflicts. The right approach is to get the
> out-of-tree driver upstream.

I'm happy to keep it in.

When the MicroSemi driver does eventually go upstream this will
probably have to be discussed though as either the config or device
tree will need to be edited to ensure only one PCIe device is present.

>
> Enabling general support for PCIe in the defconfig should be fine,
> though.

Ok, I'll send a v2 with the Xilinx PCIe still enabled.

Alistair

>
>
> - Paul
>