Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752005AbdGaW62 (ORCPT ); Mon, 31 Jul 2017 18:58:28 -0400 Received: from mail.kernel.org ([198.145.29.99]:52328 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751978AbdGaW6Y (ORCPT ); Mon, 31 Jul 2017 18:58:24 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org B349022BEA Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=helgaas@kernel.org Date: Mon, 31 Jul 2017 17:58:22 -0500 From: Bjorn Helgaas To: Paul Burton Cc: Guenter Roeck , Bjorn Helgaas , linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Michal Simek , =?iso-8859-1?Q?S=F6ren?= Brinkmann , James Hogan Subject: Re: [PATCH] PCI: xilinx: Remove platform/architecture restrictions Message-ID: <20170731225822.GB19800@bhelgaas-glaptop.roam.corp.google.com> References: <1500856777-23383-1-git-send-email-linux@roeck-us.net> <3882569.bb2rKKAOIY@np-p-burton> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <3882569.bb2rKKAOIY@np-p-burton> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2488 Lines: 64 On Mon, Jul 24, 2017 at 11:49:22AM +0100, Paul Burton wrote: > Hi Guenter & all, > > On Monday, 24 July 2017 01:39:37 BST Guenter Roeck wrote: > > The MIPS Boston board configuration tries to enable CONFIG_PCIE_XILINX. > > That doesn't work since PCIE_XILINX depends on ARCH_ZYNQ || MICROBLAZE. > > Remove that restriction. > > I'd prefer that this patch does not go in standalone. The intent for the MIPS > Boston board is that this driver is enabled for MIPS by this patch: > > https://patchwork.kernel.org/patch/9794361/ > > But not until after earlier patches in that series fix issues with the driver: > > https://patchwork.kernel.org/patch/9794355/ > https://patchwork.kernel.org/patch/9794357/ > https://patchwork.kernel.org/patch/9794359/ > > That has been held up by disagreement about whether the driver should be using > 0-3 or 1-4 for hardware IRQ numbers, sadly, despite the driver already being > in tree & clearly broken, and my series not changing which the driver uses... It's true that your v5 series only changes xilinx from using hwirq 0-3 to 0-4 (with 0 being unused in both cases, and the addition of 4 fixing the "INTD doesn't work" bug). However, I *would* like to see this issue cleaned up consistently across all our drivers. I mooted a couple ideas in [1], but nobody seemed interested. If I merged your series as-is, there would be even less interest. [1] http://lkml.kernel.org/r/20170712221455.GJ14614@bhelgaas-glaptop.roam.corp.google.com > In any case, I don't really mind if people would rather remove the > architecture restrictions than just add MIPS, but I'd prefer this doesn't go > in until the rest of my series since without at least patch 1 of my seres this > will lead to various WARN()s on Boston boards. > > Thanks, > Paul > > > > > Cc: Paul Burton > > Cc: James Hogan > > Signed-off-by: Guenter Roeck > > --- > > drivers/pci/host/Kconfig | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/drivers/pci/host/Kconfig b/drivers/pci/host/Kconfig > > index 89d61c2cbfaa..ed905a5401c3 100644 > > --- a/drivers/pci/host/Kconfig > > +++ b/drivers/pci/host/Kconfig > > @@ -71,7 +71,6 @@ config PCI_HOST_GENERIC > > > > config PCIE_XILINX > > bool "Xilinx AXI PCIe host bridge support" > > - depends on ARCH_ZYNQ || MICROBLAZE > > help > > Say 'Y' here if you want kernel to support the Xilinx AXI PCIe > > Host Bridge driver. >