From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756129AbbCMOsv (ORCPT ); Fri, 13 Mar 2015 10:48:51 -0400 Received: from mail-qc0-f174.google.com ([209.85.216.174]:34838 "EHLO mail-qc0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752324AbbCMOsq (ORCPT ); Fri, 13 Mar 2015 10:48:46 -0400 MIME-Version: 1.0 In-Reply-To: <20150313113156.GA4739@bart.dudau.co.uk> References: <1425868467-9667-1-git-send-email-wangyijing@huawei.com> <20150312212306.GH7346@google.com> <20150313113156.GA4739@bart.dudau.co.uk> From: Bjorn Helgaas Date: Fri, 13 Mar 2015 09:48:25 -0500 Message-ID: Subject: Re: [PATCH v6 00/30] Refine PCI scan interfaces and make generic pci host bridge To: Liviu Dudau Cc: Yijing Wang , Jiang Liu , "linux-pci@vger.kernel.org" , Yinghai Lu , "linux-kernel@vger.kernel.org" , Marc Zyngier , linux-arm , Russell King , "x86@kernel.org" , Thomas Gleixner , Benjamin Herrenschmidt , Rusty Russell , Tony Luck , "linux-ia64@vger.kernel.org" , "David S. Miller" , Guan Xuetao , linux-alpha@vger.kernel.org, linux-m68k@vger.kernel.org, Arnd Bergmann , Geert Uytterhoeven Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Mar 13, 2015 at 6:31 AM, Liviu Dudau wrote: > On Thu, Mar 12, 2015 at 04:23:06PM -0500, Bjorn Helgaas wrote: >> On Mon, Mar 09, 2015 at 10:33:57AM +0800, Yijing Wang wrote: >> > ... >> > Yijing Wang (29): >> > PCI: Rip out pci_bus_add_devices() from pci_scan_bus() >> > PCI: Rip out pci_bus_add_devices() from pci_scan_root_bus() >> > sparc/PCI: Claim bus resources before pci_bus_add_devices() >> >> I put the above patches plus a cleanup patch on my pci/enumeration branch. >> Please rebase your next revision to that. If there's anything else >> non-controversial that we can pull in to chip away at this, let me know. > > Hi Bjorn and Yijing, > > Sorry for not providing feedback earlier on this series (or any previous ones), > I'm just crawling out from under a rock of doing graphics drivers :) > > The patch ripping out pci_bus_add_devices() from pci_scan_root_bus() misses out > the users of that function from drivers/pci/host. If Yijing is going to refresh > the series it is probably worth starting with adding back the call into pcie-xilinx.c > and pci-versatile.c. Huh, sure enough, I missed those, too. Yijing, when you add those, can you pull my branch, update the patch, and repost it? That way you'll keep my changelog and cleanup updates. Bjorn From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-qg0-f46.google.com ([209.85.192.46]:40956 "EHLO mail-qg0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754169AbbCMOsq (ORCPT ); Fri, 13 Mar 2015 10:48:46 -0400 Received: by qgdq107 with SMTP id q107so26348845qgd.7 for ; Fri, 13 Mar 2015 07:48:46 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <20150313113156.GA4739@bart.dudau.co.uk> References: <1425868467-9667-1-git-send-email-wangyijing@huawei.com> <20150312212306.GH7346@google.com> <20150313113156.GA4739@bart.dudau.co.uk> From: Bjorn Helgaas Date: Fri, 13 Mar 2015 09:48:25 -0500 Message-ID: Subject: Re: [PATCH v6 00/30] Refine PCI scan interfaces and make generic pci host bridge To: Liviu Dudau Cc: Yijing Wang , Jiang Liu , "linux-pci@vger.kernel.org" , Yinghai Lu , "linux-kernel@vger.kernel.org" , Marc Zyngier , linux-arm , Russell King , "x86@kernel.org" , Thomas Gleixner , Benjamin Herrenschmidt , Rusty Russell , Tony Luck , "linux-ia64@vger.kernel.org" , "David S. Miller" , Guan Xuetao , linux-alpha@vger.kernel.org, linux-m68k@lists.linux-m68k.org, Arnd Bergmann , Geert Uytterhoeven Content-Type: text/plain; charset=UTF-8 Sender: linux-pci-owner@vger.kernel.org List-ID: On Fri, Mar 13, 2015 at 6:31 AM, Liviu Dudau wrote: > On Thu, Mar 12, 2015 at 04:23:06PM -0500, Bjorn Helgaas wrote: >> On Mon, Mar 09, 2015 at 10:33:57AM +0800, Yijing Wang wrote: >> > ... >> > Yijing Wang (29): >> > PCI: Rip out pci_bus_add_devices() from pci_scan_bus() >> > PCI: Rip out pci_bus_add_devices() from pci_scan_root_bus() >> > sparc/PCI: Claim bus resources before pci_bus_add_devices() >> >> I put the above patches plus a cleanup patch on my pci/enumeration branch. >> Please rebase your next revision to that. If there's anything else >> non-controversial that we can pull in to chip away at this, let me know. > > Hi Bjorn and Yijing, > > Sorry for not providing feedback earlier on this series (or any previous ones), > I'm just crawling out from under a rock of doing graphics drivers :) > > The patch ripping out pci_bus_add_devices() from pci_scan_root_bus() misses out > the users of that function from drivers/pci/host. If Yijing is going to refresh > the series it is probably worth starting with adding back the call into pcie-xilinx.c > and pci-versatile.c. Huh, sure enough, I missed those, too. Yijing, when you add those, can you pull my branch, update the patch, and repost it? That way you'll keep my changelog and cleanup updates. Bjorn From mboxrd@z Thu Jan 1 00:00:00 1970 From: bhelgaas@google.com (Bjorn Helgaas) Date: Fri, 13 Mar 2015 09:48:25 -0500 Subject: [PATCH v6 00/30] Refine PCI scan interfaces and make generic pci host bridge In-Reply-To: <20150313113156.GA4739@bart.dudau.co.uk> References: <1425868467-9667-1-git-send-email-wangyijing@huawei.com> <20150312212306.GH7346@google.com> <20150313113156.GA4739@bart.dudau.co.uk> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, Mar 13, 2015 at 6:31 AM, Liviu Dudau wrote: > On Thu, Mar 12, 2015 at 04:23:06PM -0500, Bjorn Helgaas wrote: >> On Mon, Mar 09, 2015 at 10:33:57AM +0800, Yijing Wang wrote: >> > ... >> > Yijing Wang (29): >> > PCI: Rip out pci_bus_add_devices() from pci_scan_bus() >> > PCI: Rip out pci_bus_add_devices() from pci_scan_root_bus() >> > sparc/PCI: Claim bus resources before pci_bus_add_devices() >> >> I put the above patches plus a cleanup patch on my pci/enumeration branch. >> Please rebase your next revision to that. If there's anything else >> non-controversial that we can pull in to chip away at this, let me know. > > Hi Bjorn and Yijing, > > Sorry for not providing feedback earlier on this series (or any previous ones), > I'm just crawling out from under a rock of doing graphics drivers :) > > The patch ripping out pci_bus_add_devices() from pci_scan_root_bus() misses out > the users of that function from drivers/pci/host. If Yijing is going to refresh > the series it is probably worth starting with adding back the call into pcie-xilinx.c > and pci-versatile.c. Huh, sure enough, I missed those, too. Yijing, when you add those, can you pull my branch, update the patch, and repost it? That way you'll keep my changelog and cleanup updates. Bjorn From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bjorn Helgaas Date: Fri, 13 Mar 2015 14:48:25 +0000 Subject: Re: [PATCH v6 00/30] Refine PCI scan interfaces and make generic pci host bridge Message-Id: List-Id: References: <1425868467-9667-1-git-send-email-wangyijing@huawei.com> <20150312212306.GH7346@google.com> <20150313113156.GA4739@bart.dudau.co.uk> In-Reply-To: <20150313113156.GA4739@bart.dudau.co.uk> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Liviu Dudau Cc: Yijing Wang , Jiang Liu , "linux-pci@vger.kernel.org" , Yinghai Lu , "linux-kernel@vger.kernel.org" , Marc Zyngier , linux-arm , Russell King , "x86@kernel.org" , Thomas Gleixner , Benjamin Herrenschmidt , Rusty Russell , Tony Luck , "linux-ia64@vger.kernel.org" , "David S. Miller" , Guan Xuetao , linux-alpha@vger.kernel.org, linux-m68k@vger.kernel.org, Arnd Bergmann , Geert Uytterhoeven On Fri, Mar 13, 2015 at 6:31 AM, Liviu Dudau wrote: > On Thu, Mar 12, 2015 at 04:23:06PM -0500, Bjorn Helgaas wrote: >> On Mon, Mar 09, 2015 at 10:33:57AM +0800, Yijing Wang wrote: >> > ... >> > Yijing Wang (29): >> > PCI: Rip out pci_bus_add_devices() from pci_scan_bus() >> > PCI: Rip out pci_bus_add_devices() from pci_scan_root_bus() >> > sparc/PCI: Claim bus resources before pci_bus_add_devices() >> >> I put the above patches plus a cleanup patch on my pci/enumeration branch. >> Please rebase your next revision to that. If there's anything else >> non-controversial that we can pull in to chip away at this, let me know. > > Hi Bjorn and Yijing, > > Sorry for not providing feedback earlier on this series (or any previous ones), > I'm just crawling out from under a rock of doing graphics drivers :) > > The patch ripping out pci_bus_add_devices() from pci_scan_root_bus() misses out > the users of that function from drivers/pci/host. If Yijing is going to refresh > the series it is probably worth starting with adding back the call into pcie-xilinx.c > and pci-versatile.c. Huh, sure enough, I missed those, too. Yijing, when you add those, can you pull my branch, update the patch, and repost it? That way you'll keep my changelog and cleanup updates. Bjorn