On Thu, Dec 01, 2016 at 03:34:52PM -0500, Andy Gospodarek wrote: > PAXC and PAXCv2 buses do not support legacy IRQs so there is no reason > to even try and map them. Without a change like this, one cannot create > VFs on Nitro ports since legacy interrupts are checked as part of the > PCI device creation process. Testing on PAXC hardware showed that VFs > are properly created with only the change to not set pcie->map_irq, but > just to be safe the change in iproc_pcie_setup will ensure that > pdev_fixup_irq will not panic. > > Signed-off-by: Andy Gospodarek <gospo@xxxxxxxxxxxx> > Signed-off-by: Ray Jui <rjui@xxxxxxxxxxxx> Applied to pci/host-iproc for v4.10 with Ray's acked-by, thanks! > --- > drivers/pci/host/pcie-iproc-platform.c | 9 ++++++++- > drivers/pci/host/pcie-iproc.c | 5 ++++- > 2 files changed, 12 insertions(+), 2 deletions(-) > > diff --git a/drivers/pci/host/pcie-iproc-platform.c b/drivers/pci/host/pcie-iproc-platform.c > index fd3ed9b..22d814a 100644 > --- a/drivers/pci/host/pcie-iproc-platform.c > +++ b/drivers/pci/host/pcie-iproc-platform.c > @@ -108,7 +108,14 @@ static int iproc_pcie_pltfm_probe(struct platform_device *pdev) > return ret; > } > > - pcie->map_irq = of_irq_parse_and_map_pci; > + /* PAXC doesn't support legacy IRQs, skip mapping */ > + switch (pcie->type) { > + case IPROC_PCIE_PAXC: > + case IPROC_PCIE_PAXC_V2: > + break; > + default: > + pcie->map_irq = of_irq_parse_and_map_pci; > + } > > ret = iproc_pcie_setup(pcie, &res); > if (ret) > diff --git a/drivers/pci/host/pcie-iproc.c b/drivers/pci/host/pcie-iproc.c > index cd51334..3ebc025 100644 > --- a/drivers/pci/host/pcie-iproc.c > +++ b/drivers/pci/host/pcie-iproc.c > @@ -1274,7 +1274,10 @@ int iproc_pcie_setup(struct iproc_pcie *pcie, struct list_head *res) > > pci_scan_child_bus(bus); > pci_assign_unassigned_bus_resources(bus); > - pci_fixup_irqs(pci_common_swizzle, pcie->map_irq); > + > + if (pcie->map_irq) > + pci_fixup_irqs(pci_common_swizzle, pcie->map_irq); > + > pci_bus_add_devices(bus); > > return 0; > -- > 2.1.0 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html