Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752963AbaKVC6t (ORCPT ); Fri, 21 Nov 2014 21:58:49 -0500 Received: from mail-pa0-f46.google.com ([209.85.220.46]:33770 "EHLO mail-pa0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752530AbaKVC6r (ORCPT ); Fri, 21 Nov 2014 21:58:47 -0500 Message-ID: <546FFBBD.8060909@gmail.com> Date: Sat, 22 Nov 2014 10:58:05 +0800 From: Yijing Wang User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: Bjorn Helgaas , Yijing Wang , g@google.com CC: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Xinwei Hu , Wuyun , linux-arm-kernel@lists.infradead.org, Russell King , Thomas Gleixner , Thierry Reding , Thomas Petazzoni , Marc Zyngier Subject: Re: [PATCH 00/10] Save MSI chip in pci_sys_data References: <1414396127-30023-1-git-send-email-wangyijing@huawei.com> <20141112042359.GK28161@google.com> <20141121172018.GA6578@google.com> In-Reply-To: <20141121172018.GA6578@google.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 在 2014/11/22 1:20, Bjorn Helgaas 写道: > [+cc Marc] > > On Tue, Nov 11, 2014 at 09:23:59PM -0700, Bjorn Helgaas wrote: >> On Mon, Oct 27, 2014 at 03:48:37PM +0800, Yijing Wang wrote: >>> Now PCI host bridge drivers in arm associate MSI chip and >>> PCI bus by adding .add_bus(), and assign MSI chip pointer >>> to every PCI bus. Associating MSI chip and every PCI bus >>> is not necessary. All PCI busses under same PCI host brdige >>> share the same MSI chip. So saving MSI chip in pci_sys_data >>> is a better solution, it make PCI host bridge drivers clean. >>> Because we still need to provide arch spec pcibios_msi_controller() >>> to extract MSI controller pointer, a better solution is to >>> refactor PCI host bridge, make a generic pci_host_bridge, and >>> save common info like PCI domain number, MSI chip, resources >>> in it. We will do that work in another series as soon. >>> >>> To Bjorn: Because struct msi_chip defined in struct hw_pci and pci_sys_data >>> is under the #ifdef CONFIG_PCI_MSI, if we use if(IS_ENABLED(CONFIG_PCI_MSI)) >>> in PCI host bridge drivers, it will cause build errors when the CONFIG_PCI_MSI >>> is off. So I keep #ifdef CONFIG_PCI_MSI in this series. >>> >>> Yijing Wang (10): >>> MSI: Rename msi_chip to msi_controller for better readability >>> PCI/MSI: Introduce weak pcibios_msi_controller() >>> arm/MSI: Save MSI controller in pci_sys_data >>> PCI: tegra: Save MSI controller in pci_sys_data >>> PCI: designware: Save MSI controller in pci_sys_data >>> PCI: rcar: Save MSI controller in pci_sys_data >>> PCI: mvebu: Save MSI controller in pci_sys_data >>> PCI: xilinx: Save MSI controller in pci_sys_data >>> arm/PCI: Clean unused pcibios_add_bus() and pcibios_remove_bus() >>> PCI/MSI: Remove useless bus->msi assignment >>> >>> arch/arm/include/asm/mach/pci.h | 10 +++++--- >>> arch/arm/kernel/bios32.c | 28 ++++++++++-------------- >>> drivers/irqchip/irq-armada-370-xp.c | 22 +++++++++--------- >>> drivers/of/of_pci.c | 40 +++++++++++++++++----------------- >>> drivers/pci/host/pci-keystone-dw.c | 4 +- >>> drivers/pci/host/pci-keystone.h | 2 +- >>> drivers/pci/host/pci-mvebu.c | 14 ++++------- >>> drivers/pci/host/pci-tegra.c | 37 +++++++++++++------------------- >>> drivers/pci/host/pcie-designware.c | 25 +++++++-------------- >>> drivers/pci/host/pcie-designware.h | 2 +- >>> drivers/pci/host/pcie-rcar.c | 37 +++++++++++++------------------- >>> drivers/pci/host/pcie-xilinx.c | 27 +++++++---------------- >>> drivers/pci/msi.c | 22 ++++++++++++++----- >>> drivers/pci/probe.c | 1 - >>> include/linux/msi.h | 6 ++-- >>> include/linux/of_pci.h | 14 ++++++------ >>> include/linux/pci.h | 2 +- >>> 17 files changed, 132 insertions(+), 161 deletions(-) >>> >> Applied to pci/msi for v3.19, thanks. > I reworked this series slightly to: > > - Change pci_msi_controller() and pcibios_msi_controller() from taking a > pci_bus * to taking a pci_dev *. This is so the interface allows > per-device MSI controllers. I don't think there's any reason to assume > all devices on a bus have to have the same controller. > > - Drop the last patch ("PCI/MSI: Remove useless bus->msi assignment") > because it broke Marc's follow-on patches. > > I updated pci/msi, but I haven't put it in my "next" branch yet because I > want confirmation from Fengguang's autobuilder that I didn't break > anything. > > Current status of my tree: > > 6cf00af0ae15 [1] pci/msi (contains this rework) > 149792795d2b [2] next (does *not* contain this rework) > > Thomas, if pci/msi looks good to you, feel free to pull it into your tree. > The only change you should need to make is to change the parameters to > pci_msi_controller() and pcibios_msi_controller(). > > Just FYI, I'm leaving on vacation for a week, so I won't be able to fix any > issues until Dec 1. Hi Bjorn, thanks very much for your improvement for this series. It looks good to me. :) > > Bjorn > > [1] https://git.kernel.org/cgit/linux/kernel/git/helgaas/pci.git/log/?h=next&id=6cf00af0ae15 > [2] https://git.kernel.org/cgit/linux/kernel/git/helgaas/pci.git/log/?h=next&id=149792795d2b > -- > To unsubscribe from this list: send the line "unsubscribe linux-pci" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/