From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752890AbdK1Nq3 (ORCPT ); Tue, 28 Nov 2017 08:46:29 -0500 Received: from Galois.linutronix.de ([146.0.238.70]:33086 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752453AbdK1Nq2 (ORCPT ); Tue, 28 Nov 2017 08:46:28 -0500 Date: Tue, 28 Nov 2017 14:45:55 +0100 (CET) From: Thomas Gleixner To: Sinan Kaya cc: linux-pci@vger.kernel.org, timur@codeaurora.org, linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, intel-gfx@lists.freedesktop.org, Bjorn Helgaas , Ingo Molnar , "H. Peter Anvin" , "maintainer:X86 ARCHITECTURE (32-BIT AND 64-BIT)" , "open list:X86 ARCHITECTURE (32-BIT AND 64-BIT)" Subject: Re: [PATCH V3 03/29] x86/PCI: deprecate pci_get_bus_and_slot() In-Reply-To: <1511801886-6753-4-git-send-email-okaya@codeaurora.org> Message-ID: References: <1511801886-6753-1-git-send-email-okaya@codeaurora.org> <1511801886-6753-4-git-send-email-okaya@codeaurora.org> User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 27 Nov 2017, Sinan Kaya wrote: > pci_get_bus_and_slot() is restrictive such that it assumes domain=0 as > where a PCI device is present. This restricts the device drivers to be > reused for other domain numbers. > > Getting ready to remove pci_get_bus_and_slot() function in favor of > pci_get_domain_bus_and_slot(). > > Use domain number of 0 as the domain number is not available in struct > irq_routing_table. > > Signed-off-by: Sinan Kaya Acked-by: Thomas Gleixner > --- > arch/x86/pci/irq.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/arch/x86/pci/irq.c b/arch/x86/pci/irq.c > index 04526291..52e5510 100644 > --- a/arch/x86/pci/irq.c > +++ b/arch/x86/pci/irq.c > @@ -839,7 +839,8 @@ static void __init pirq_find_router(struct irq_router *r) > DBG(KERN_DEBUG "PCI: Attempting to find IRQ router for [%04x:%04x]\n", > rt->rtr_vendor, rt->rtr_device); > > - pirq_router_dev = pci_get_bus_and_slot(rt->rtr_bus, rt->rtr_devfn); > + pirq_router_dev = pci_get_domain_bus_and_slot(0, rt->rtr_bus, > + rt->rtr_devfn); > if (!pirq_router_dev) { > DBG(KERN_DEBUG "PCI: Interrupt router not found at " > "%02x:%02x\n", rt->rtr_bus, rt->rtr_devfn); > -- > 1.9.1 > >