irq_create_strict_mappings() is a poor way to allow the use of
a linear IRQ domain as a legacy one. Let's be upfront about
it and use a legacy domain when appropriate.

Signed-off-by: Marc Zyngier <m...@kernel.org>
---
 arch/arm/mach-pxa/pxa_cplds_irqs.c | 24 +++++++++++-------------
 1 file changed, 11 insertions(+), 13 deletions(-)

diff --git a/arch/arm/mach-pxa/pxa_cplds_irqs.c 
b/arch/arm/mach-pxa/pxa_cplds_irqs.c
index 45c19ca96f7a..ec0d9b094744 100644
--- a/arch/arm/mach-pxa/pxa_cplds_irqs.c
+++ b/arch/arm/mach-pxa/pxa_cplds_irqs.c
@@ -147,22 +147,20 @@ static int cplds_probe(struct platform_device *pdev)
        }
 
        irq_set_irq_wake(fpga->irq, 1);
-       fpga->irqdomain = irq_domain_add_linear(pdev->dev.of_node,
-                                              CPLDS_NB_IRQ,
-                                              &cplds_irq_domain_ops, fpga);
+       if (base_irq)
+               fpga->irqdomain = irq_domain_add_legacy(pdev->dev.of_node,
+                                                       CPLDS_NB_IRQ,
+                                                       base_irq, 0,
+                                                       &cplds_irq_domain_ops,
+                                                       fpga);
+       else
+               fpga->irqdomain = irq_domain_add_linear(pdev->dev.of_node,
+                                                       CPLDS_NB_IRQ,
+                                                       &cplds_irq_domain_ops,
+                                                       fpga);
        if (!fpga->irqdomain)
                return -ENODEV;
 
-       if (base_irq) {
-               ret = irq_create_strict_mappings(fpga->irqdomain, base_irq, 0,
-                                                CPLDS_NB_IRQ);
-               if (ret) {
-                       dev_err(&pdev->dev, "couldn't create the irq mapping 
%d..%d\n",
-                               base_irq, base_irq + CPLDS_NB_IRQ);
-                       return ret;
-               }
-       }
-
        return 0;
 }
 
-- 
2.29.2

Reply via email to