On Thu, Dec 07, 2017 at 03:14:22PM +0000, Joao Pinto wrote: > Hello to all, > > I am sending this e-mail to request your opinion about the tasks needed to add > support for MSI-X for pcie-designware based solutions. > > A few months ago I submited a patch-set that added support for MSI-X: > > [v2,9/9] pci: remove limitation of the number of the available IRQs > https://patchwork.ozlabs.org/patch/771320/ > [v2,8/9] pci: removing old irq api from pcie-designware > https://patchwork.ozlabs.org/patch/771360/ > [v2,7/9] pci: keystone SoC driver adapted to new irq API > https://patchwork.ozlabs.org/patch/771361/ > [v2,6/9] pci: qcom SoC driver adapted to new irq API > https://patchwork.ozlabs.org/patch/771362/ > [v2,5/9] pci: generic PCIe DW driver adapted to new irq API > https://patchwork.ozlabs.org/patch/771365/ > [v2,4/9] pci: artpec6 SoC driver adapted to new irq API > https://patchwork.ozlabs.org/patch/771364/ > [v2,3/9] pci: imx6 SoC driver adapted to new irq API > https://patchwork.ozlabs.org/patch/771363/ > [v2,2/9] pci: exynos SoC driver adapted to new irq API > https://patchwork.ozlabs.org/patch/771359/ > [v2,1/9] pci: adding new irq api to pci-designware > https://patchwork.ozlabs.org/patch/771366/ > > The patch-set was globally accepted, but it broke the MSI mechanism for > Keystone, due to its specificity. Hi Joao, would you mind pointing me at the discussion where the breakage was explained please ? It is to understand what are the TI specific bits we are talking about here. > We are now going to resume this task and we would like to have your feedback > about our plan: > > Task 1: Help TI to create a keystone_msi driver to isolate its custom msi mechanism > Task 2: Port the existing patches to the new kernel version (except the keystone > one) > > If TI can do the keystone_msi implementation it would be easier. If it's not > possible, we volunteer to do it, but we will need testing & debug assistance. I would ask Murali to chime in please to understand the direction to take - at least I expect him to be able to test the changes. Thanks, Lorenzo
