linux-riscv.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: Alistair Francis <Alistair.Francis@wdc.com>
To: "paul.walmsley@sifive.com" <paul.walmsley@sifive.com>
Cc: "linux-riscv@lists.infradead.org"
	<linux-riscv@lists.infradead.org>,
	"palmer@sifive.com" <palmer@sifive.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] RISC-V: defconfig: Enable Generic PCIE by default
Date: Fri, 21 Dec 2018 19:22:41 +0000	[thread overview]
Message-ID: <541e8b85bc2d3010741daadddc71ada52b847b8f.camel@wdc.com> (raw)
In-Reply-To: <alpine.DEB.2.21.9999.1812211110260.777@viisi.sifive.com>

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
> 
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv

  reply	other threads:[~2018-12-21 19:23 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-12-21 19:06 [PATCH] RISC-V: defconfig: Enable Generic PCIE by default Alistair Francis
2018-12-21 19:13 ` Paul Walmsley
2018-12-21 19:22   ` Alistair Francis [this message]
2018-12-21 19:27     ` Paul Walmsley
2018-12-21 22:11       ` Alistair Francis
2018-12-21 22:34         ` Paul Walmsley
2018-12-21 23:03           ` Alistair Francis
2018-12-26 16:52       ` Palmer Dabbelt

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=541e8b85bc2d3010741daadddc71ada52b847b8f.camel@wdc.com \
    --to=alistair.francis@wdc.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-riscv@lists.infradead.org \
    --cc=palmer@sifive.com \
    --cc=paul.walmsley@sifive.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).