lkml.org 
[lkml]   [2015]   [Sep]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH v2 04/12] drivers:pci: Add IRQ domain lookup by PCI domain
Hi Jake,

I don't know how this will be morphed after Marc's comments, but when you
repost this series, please take a look at the existing change history,
e.g., with "git log --oneline drivers/pci/probe.c", and make your subject
lines follow the existing conventions. In this case, they look like:

PCI: Make pci_msi_setup_pci_dev() non-static for use by arch code
PCI: Set MPS to match upstream bridge
PCI: Move MPS configuration check to pci_configure_device()
PCI: Add pci_scan_root_bus_msi()
PCI: Tolerate hierarchies with no Root Port
...

Bjorn

On Fri, Sep 11, 2015 at 12:01:03AM +0000, jakeo@microsoft.com wrote:
> From: Jake Oshins <jakeo@microsoft.com>
>
> The PCI driver currently looks up IRQ domains for root PCI buses by walking
> up the Open Firmware tree looking for any that cover this particular PCI root.
> Since x86 PCs don't implement Open Firmware, this patch offers an alternative
> lookup by the PCI domain ID (known as "segment" in the PCI and ACPI specs.)
>
> I could have tried to build a (fake) Open Firmware tree and leverage the old
> code, but I rejected that possibility both because it would have required
> changes in lots of other places and because most distributions don't even
> compile in the OF infrastructure when targeting PCs.
>
> Signed-off-by: Jake Oshins <jakeo@microsoft.com>
> ---
> drivers/pci/probe.c | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/drivers/pci/probe.c b/drivers/pci/probe.c
> index 0b2be17..e7e5ff3 100644
> --- a/drivers/pci/probe.c
> +++ b/drivers/pci/probe.c
> @@ -12,6 +12,7 @@
> #include <linux/module.h>
> #include <linux/cpumask.h>
> #include <linux/pci-aspm.h>
> +#include <linux/irqdomain.h>
> #include <asm-generic/pci-bridge.h>
> #include "pci.h"
>
> @@ -663,12 +664,22 @@ static void pci_set_bus_speed(struct pci_bus *bus)
> static struct irq_domain *pci_host_bridge_msi_domain(struct pci_bus *bus)
> {
> struct irq_domain *d;
> + int pci_domain;
>
> /*
> * Any firmware interface that can resolve the msi_domain
> * should be called from here.
> */
> d = pci_host_bridge_of_msi_domain(bus);
> + if (d)
> + return d;
> +
> + /*
> + * If firmware couldn't help find, it try looking it up by PCI
> + * domain/segment.
> + */
> + pci_domain = pci_domain_nr(bus);
> + d = irq_find_matching_host(NULL, DOMAIN_BUS_PCI_MSI, &pci_domain);
>
> return d;
> }
> --
> 1.9.1
>


\
 
 \ /
  Last update: 2015-09-21 18:01    [W:0.096 / U:0.724 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site