From a522ee0199d5d3ea114ca2e211f6ac398d3e8e0b Mon Sep 17 00:00:00 2001 From: John Crispin Date: Sat, 23 Jun 2018 15:07:37 +0200 Subject: [PATCH 20/33] MIPS: pci-ar724x: convert to OF With the ath79 target getting converted to pure OF, we can drop all the platform data code and add the missing OF bits to the driver. We also add a irq domain for the PCI/e controllers cascade, thus making it usable from dts files. Signed-off-by: John Crispin --- arch/mips/pci/pci-ar724x.c | 88 ++++++++++++++++++++++------------------------ 1 file changed, 42 insertions(+), 46 deletions(-) --- a/arch/mips/pci/pci-ar724x.c +++ b/arch/mips/pci/pci-ar724x.c @@ -11,8 +11,11 @@ #include #include #include +#include #include #include +#include +#include #define AR724X_PCI_REG_APP 0x00 #define AR724X_PCI_REG_RESET 0x18 @@ -42,17 +45,20 @@ struct ar724x_pci_controller { void __iomem *crp_base; int irq; - int irq_base; bool link_up; bool bar0_is_cached; u32 bar0_value; + struct device_node *np; struct pci_controller pci_controller; + struct irq_domain *domain; struct resource io_res; struct resource mem_res; }; +static struct irq_chip ar724x_pci_irq_chip; + static inline bool ar724x_pci_check_link(struct ar724x_pci_controller *apc) { u32 reset; @@ -228,35 +234,31 @@ static struct pci_ops ar724x_pci_ops = { static void ar724x_pci_irq_handler(struct irq_desc *desc) { - struct ar724x_pci_controller *apc; - void __iomem *base; + struct irq_chip *chip = irq_desc_get_chip(desc); + struct ar724x_pci_controller *apc = irq_desc_get_handler_data(desc); u32 pending; - apc = irq_desc_get_handler_data(desc); - base = apc->ctrl_base; - - pending = __raw_readl(base + AR724X_PCI_REG_INT_STATUS) & - __raw_readl(base + AR724X_PCI_REG_INT_MASK); + chained_irq_enter(chip, desc); + pending = __raw_readl(apc->ctrl_base + AR724X_PCI_REG_INT_STATUS) & + __raw_readl(apc->ctrl_base + AR724X_PCI_REG_INT_MASK); if (pending & AR724X_PCI_INT_DEV0) - generic_handle_irq(apc->irq_base + 0); - + generic_handle_irq(irq_linear_revmap(apc->domain, 1)); else spurious_interrupt(); + chained_irq_exit(chip, desc); } static void ar724x_pci_irq_unmask(struct irq_data *d) { struct ar724x_pci_controller *apc; void __iomem *base; - int offset; u32 t; apc = irq_data_get_irq_chip_data(d); base = apc->ctrl_base; - offset = apc->irq_base - d->irq; - switch (offset) { + switch (irq_linear_revmap(apc->domain, d->irq)) { case 0: t = __raw_readl(base + AR724X_PCI_REG_INT_MASK); __raw_writel(t | AR724X_PCI_INT_DEV0, @@ -270,14 +272,12 @@ static void ar724x_pci_irq_mask(struct i { struct ar724x_pci_controller *apc; void __iomem *base; - int offset; u32 t; apc = irq_data_get_irq_chip_data(d); base = apc->ctrl_base; - offset = apc->irq_base - d->irq; - switch (offset) { + switch (irq_linear_revmap(apc->domain, d->irq)) { case 0: t = __raw_readl(base + AR724X_PCI_REG_INT_MASK); __raw_writel(t & ~AR724X_PCI_INT_DEV0, @@ -302,26 +302,34 @@ static struct irq_chip ar724x_pci_irq_ch .irq_mask_ack = ar724x_pci_irq_mask, }; +static int ar724x_pci_irq_map(struct irq_domain *d, + unsigned int irq, irq_hw_number_t hw) +{ + struct ar724x_pci_controller *apc = d->host_data; + + irq_set_chip_and_handler(irq, &ar724x_pci_irq_chip, handle_level_irq); + irq_set_chip_data(irq, apc); + + return 0; +} + +static const struct irq_domain_ops ar724x_pci_domain_ops = { + .xlate = irq_domain_xlate_onecell, + .map = ar724x_pci_irq_map, +}; + static void ar724x_pci_irq_init(struct ar724x_pci_controller *apc, int id) { void __iomem *base; - int i; base = apc->ctrl_base; __raw_writel(0, base + AR724X_PCI_REG_INT_MASK); __raw_writel(0, base + AR724X_PCI_REG_INT_STATUS); - apc->irq_base = ATH79_PCI_IRQ_BASE + (id * AR724X_PCI_IRQ_COUNT); - - for (i = apc->irq_base; - i < apc->irq_base + AR724X_PCI_IRQ_COUNT; i++) { - irq_set_chip_and_handler(i, &ar724x_pci_irq_chip, - handle_level_irq); - irq_set_chip_data(i, apc); - } - + apc->domain = irq_domain_add_linear(apc->np, 2, + &ar724x_pci_domain_ops, apc); irq_set_chained_handler_and_data(apc->irq, ar724x_pci_irq_handler, apc); } @@ -360,7 +368,6 @@ static void ar724x_pci_hw_init(struct ar static int ar724x_pci_probe(struct platform_device *pdev) { struct ar724x_pci_controller *apc; - struct resource *res; int id; id = pdev->id; @@ -388,29 +395,11 @@ static int ar724x_pci_probe(struct platf if (apc->irq < 0) return -EINVAL; - res = platform_get_resource_byname(pdev, IORESOURCE_IO, "io_base"); - if (!res) - return -EINVAL; - - apc->io_res.parent = res; - apc->io_res.name = "PCI IO space"; - apc->io_res.start = res->start; - apc->io_res.end = res->end; - apc->io_res.flags = IORESOURCE_IO; - - res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "mem_base"); - if (!res) - return -EINVAL; - - apc->mem_res.parent = res; - apc->mem_res.name = "PCI memory space"; - apc->mem_res.start = res->start; - apc->mem_res.end = res->end; - apc->mem_res.flags = IORESOURCE_MEM; - + apc->np = pdev->dev.of_node; apc->pci_controller.pci_ops = &ar724x_pci_ops; apc->pci_controller.io_resource = &apc->io_res; apc->pci_controller.mem_resource = &apc->mem_res; + pci_load_of_ranges(&apc->pci_controller, pdev->dev.of_node); /* * Do the full PCIE Root Complex Initialization Sequence if the PCIe @@ -432,10 +421,16 @@ static int ar724x_pci_probe(struct platf return 0; } +static const struct of_device_id ar724x_pci_ids[] = { + { .compatible = "qcom,ar7240-pci" }, + {}, +}; + static struct platform_driver ar724x_pci_driver = { .probe = ar724x_pci_probe, .driver = { .name = "ar724x-pci", + .of_match_table = of_match_ptr(ar724x_pci_ids), }, };