[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Xen-devel] [PATCH v1 15/21] Powerpc/MSI: Use MSI chip framework to configure MSI/MSI-X irq
 
- To: wangyijing <wangyijing0307@xxxxxxxxx>
 
- From: Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx>
 
- Date: Fri, 05 Sep 2014 15:41:40 +0400
 
- Cc: "linux-mips@xxxxxxxxxxxxxx" <linux-mips@xxxxxxxxxxxxxx>,	"linux-ia64@xxxxxxxxxxxxxxx" <linux-ia64@xxxxxxxxxxxxxxx>,	"linux-pci@xxxxxxxxxxxxxxx" <linux-pci@xxxxxxxxxxxxxxx>,	"Bharat.Bhushan@xxxxxxxxxxxxx" <Bharat.Bhushan@xxxxxxxxxxxxx>,	"sparclinux@xxxxxxxxxxxxxxx" <sparclinux@xxxxxxxxxxxxxxx>,	Yijing Wang <wangyijing@xxxxxxxxxx>,	"linux-arch@xxxxxxxxxxxxxxx" <linux-arch@xxxxxxxxxxxxxxx>,	"linux-s390@xxxxxxxxxxxxxxx" <linux-s390@xxxxxxxxxxxxxxx>,	Russell King <linux@xxxxxxxxxxxxxxxx>, Joerg Roedel <joro@xxxxxxxxxx>,	"x86@xxxxxxxxxx" <x86@xxxxxxxxxx>,	Sebastian Ott <sebott@xxxxxxxxxxxxxxxxxx>,	Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>,	"xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>,	"arnab.basu@xxxxxxxxxxxxx" <arnab.basu@xxxxxxxxxxxxx>,	Arnd Bergmann <arnd@xxxxxxxx>, Chris Metcalf <cmetcalf@xxxxxxxxxx>,	Bjorn Helgaas <bhelgaas@xxxxxxxxxx>, Thomas Gleixner <tglx@xxxxxxxxxxxxx>,	"linux-arm-kernel@xxxxxxxxxxxxxxxxxxx"	<linux-arm-kernel@xxxxxxxxxxxxxxxxxxx>,	Xinwei Hu <huxinwei@xxxxxxxxxx>, Tony Luck <tony.luck@xxxxxxxxx>,	Ralf Baechle <ralf@xxxxxxxxxxxxxx>,	"iommu@xxxxxxxxxxxxxxxxxxxxxxxxxx" <iommu@xxxxxxxxxxxxxxxxxxxxxxxxxx>,	Wuyun <wuyun.wu@xxxxxxxxxx>,	"linuxppc-dev@xxxxxxxxxxxxxxxx" <linuxppc-dev@xxxxxxxxxxxxxxxx>,	"David S. Miller" <davem@xxxxxxxxxxxxx>
 
- Delivery-date: Fri, 05 Sep 2014 11:42:02 +0000
 
- List-id: Xen developer discussion <xen-devel.lists.xen.org>
 
 
 
On 9/5/2014 3:33 PM, wangyijing wrote:
 
Use MSI chip framework instead of arch MSI functions to configure
MSI/MSI-X irq. So we can manage MSI/MSI-X irq in a unified framework.
 
 
 
 
 
Signed-off-by: Yijing Wang <wangyijing@xxxxxxxxxx>
---
  arch/powerpc/kernel/msi.c |   14 ++++++++++++--
  1 files changed, 12 insertions(+), 2 deletions(-)
 
 
 
 
 
diff --git a/arch/powerpc/kernel/msi.c b/arch/powerpc/kernel/msi.c
index 71bd161..01781a4 100644
--- a/arch/powerpc/kernel/msi.c
+++ b/arch/powerpc/kernel/msi.c
 
 
[...]
 
@@ -27,7 +27,17 @@ int arch_setup_msi_irqs(struct pci_dev *dev, int nvec, int 
type)
      return ppc_md.setup_msi_irqs(dev, nvec, type);
  }
-void arch_teardown_msi_irqs(struct pci_dev *dev)
+static void ppc_teardown_msi_irqs(struct pci_dev *dev)
 
 
 
 
   Shouldn't this function take IRQ # instead?
 
 
 
 
This function need to teardown all msi irqs of the pci dev, we should pass the 
pci dev as argument .
 
 
    Ah, I've mixed up the teardown_irqs() method with teardown_irq()! Too 
similar. :-)
 
Thanks!
Yijing.
 
 
WBR, Sergei
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel
 
 
    
     |