Dear Thierry Reding, On Mon, 15 Jul 2013 09:27:10 -0700, Thierry Reding wrote: > We need at least patches 2, 4 and 8 for Tegra. Are you sure you don't want patch 1 ? It allows to replace the allocation of MSI interrupt numbers for the MSI driver by a generic facility based on irq domains. > There's still the option > of working around patch 8 by using the custom root bus scan function > that we have to use anyway, so I guess we could even trim the > requirement down to patches 2 and 4 if we really need. Bjorn said he didn't like having the msi chip assignment in the middle of the root bus scan function, because he was hoping someday to refactor this function. Quoting Bjorn: """ > + bus->msi = pcie->msi; Ideally, arches could use pci_scan_root_bus(), which does the pci_scan_child_bus() itself, so I never like to add arch-specific code between pci_create_root_bus() and pci_scan_child_bus(). But you should be able to accomplish this by setting bus->msi in a pcibios_add_bus() hook. There is a corresponding pcibios_remove_bus() hook that might be useful for deallocating the msi_chip. I'm not too clear on how you manage the msi_chip lifetime -- I see the alloc in armada_370_xp_msi_init(), but I'm not sure whether you ever deallocate it or how you know when it would be safe to do so. """ This is from: Subject: Re: [PATCH v2 6/8] pci: mvebu: add support for MSI Date: Tue, 18 Jun 2013 16:57:11 -0600 Best regards, Thomas -- Thomas Petazzoni, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- 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
