On 07/16/2015 02:04 AM, Lorenzo Pieralisi wrote:
Hi David,
On Wed, Jul 15, 2015 at 05:54:43PM +0100, David Daney wrote:
From: David Daney <david.daney@xxxxxxxxxx>
[...]
diff --git a/arch/arm64/kernel/pci.c b/arch/arm64/kernel/pci.c
index 4095379..3356023 100644
--- a/arch/arm64/kernel/pci.c
+++ b/arch/arm64/kernel/pci.c
@@ -38,11 +38,21 @@ resource_size_t pcibios_align_resource(void *data, const struct resource *res,
return res->start;
}
+static int (*pcibios_add_device_impl)(struct pci_dev *);
+
+void set_pcibios_add_device(int (*arg)(struct pci_dev *))
+{
+ pcibios_add_device_impl = arg;
+}
+
/*
* Try to assign the IRQ number from DT when adding a new device
*/
int pcibios_add_device(struct pci_dev *dev)
{
+ if (pcibios_add_device_impl)
+ return pcibios_add_device_impl(dev);
I am totally against this (and to be honest by reading the other
patches I failed to understand why you even need it), see above.
It is because ...
Thanks,
Lorenzo
+
dev->irq = of_irq_parse_and_map_pci(dev, 0, 0);
... this is total crap. But I didn't want to break existing systems.
The PCI RC drivers need a way to configure the legacy virtual-wire
interrupts, because the existing code doesn't do it.
David Daney
--
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