[PATCH v7 2/2] PCI: mediatek: Using chained IRQ to setup IRQ handle
From: Lorenzo Pieralisi <hidden>
Date: 2018-05-21 14:03:48
Also in:
linux-devicetree, linux-mediatek, linux-pci, lkml
On Fri, May 18, 2018 at 02:51:54PM -0500, Bjorn Helgaas wrote:
quoted hunk ↗ jump to hunk
On Fri, May 04, 2018 at 01:47:33PM +0800, honghui.zhang at mediatek.com wrote:quoted
From: Honghui Zhang <redacted> Using irq_chip solution to setup IRQs in order to consist with IRQ framework. Signed-off-by: Honghui Zhang <redacted> Acked-by: Ryder Lee <ryder.lee@mediatek.com> --- drivers/pci/host/pcie-mediatek.c | 206 ++++++++++++++++++++++----------------- 1 file changed, 115 insertions(+), 91 deletions(-)diff --git a/drivers/pci/host/pcie-mediatek.c b/drivers/pci/host/pcie-mediatek.c index c3dc549..dabf1086 100644 --- a/drivers/pci/host/pcie-mediatek.c +++ b/drivers/pci/host/pcie-mediatek.c...quoted
-static int mtk_pcie_msi_map(struct irq_domain *domain, unsigned int irq, - irq_hw_number_t hwirq) +static struct msi_domain_info mtk_msi_domain_info = {I think this patch should be amended to include this:diff --git a/drivers/pci/host/Kconfig b/drivers/pci/host/Kconfig index 0d0177ce436c..368b70d9371b 100644 --- a/drivers/pci/host/Kconfig +++ b/drivers/pci/host/Kconfig@@ -193,7 +193,7 @@ config PCIE_MEDIATEK bool "MediaTek PCIe controller" depends on (ARM || ARM64) && (ARCH_MEDIATEK || COMPILE_TEST) depends on OF - depends on PCI
Actually I amended your patch, since I do not think the deletion above belongs in this commit (I agree that's completely useless but that's not this patch that we should remove it). Lorenzo
+ depends on PCI_MSI_IRQ_DOMAIN