Commit 5b8aae489a07ac7d5a2cb897d6ca1fddb0c0043a

Authored by Axel Lin
Committed by Thomas Gleixner
1 parent baaecfa724

irqchip: nvic: Fix wrong num_ct argument for irq_alloc_domain_generic_chips()

The third parameter of irq_alloc_domain_generic_chips() is the number of
irq_chip_type instances associated with these chips rather than numbanks.

Signed-off-by: Axel Lin <axel.lin@ingics.com>
Cc: Uwe Kleine-Koenig <u.kleine-koenig@pengutronix.de>
Cc: Catalin Marinas <catalin.marinas@arm.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Grant Likely <grant.likely@secretlab.ca>
Cc: kernel@pengutronix.de
Cc: linux-arm-kernel@lists.infradead.org
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>

Showing 1 changed file with 1 additions and 1 deletions Inline Diff

drivers/irqchip/irq-nvic.c
1 /* 1 /*
2 * drivers/irq/irq-nvic.c 2 * drivers/irq/irq-nvic.c
3 * 3 *
4 * Copyright (C) 2008 ARM Limited, All Rights Reserved. 4 * Copyright (C) 2008 ARM Limited, All Rights Reserved.
5 * Copyright (C) 2013 Pengutronix 5 * Copyright (C) 2013 Pengutronix
6 * 6 *
7 * This program is free software; you can redistribute it and/or modify 7 * This program is free software; you can redistribute it and/or modify
8 * it under the terms of the GNU General Public License version 2 as 8 * it under the terms of the GNU General Public License version 2 as
9 * published by the Free Software Foundation. 9 * published by the Free Software Foundation.
10 * 10 *
11 * Support for the Nested Vectored Interrupt Controller found on the 11 * Support for the Nested Vectored Interrupt Controller found on the
12 * ARMv7-M CPUs (Cortex-M3/M4) 12 * ARMv7-M CPUs (Cortex-M3/M4)
13 */ 13 */
14 #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt 14 #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
15 15
16 #include <linux/init.h> 16 #include <linux/init.h>
17 #include <linux/kernel.h> 17 #include <linux/kernel.h>
18 #include <linux/slab.h> 18 #include <linux/slab.h>
19 #include <linux/err.h> 19 #include <linux/err.h>
20 #include <linux/io.h> 20 #include <linux/io.h>
21 #include <linux/of.h> 21 #include <linux/of.h>
22 #include <linux/of_address.h> 22 #include <linux/of_address.h>
23 #include <linux/irq.h> 23 #include <linux/irq.h>
24 #include <linux/irqdomain.h> 24 #include <linux/irqdomain.h>
25 25
26 #include <asm/v7m.h> 26 #include <asm/v7m.h>
27 #include <asm/exception.h> 27 #include <asm/exception.h>
28 28
29 #include "irqchip.h" 29 #include "irqchip.h"
30 30
31 #define NVIC_ISER 0x000 31 #define NVIC_ISER 0x000
32 #define NVIC_ICER 0x080 32 #define NVIC_ICER 0x080
33 #define NVIC_IPR 0x300 33 #define NVIC_IPR 0x300
34 34
35 #define NVIC_MAX_BANKS 16 35 #define NVIC_MAX_BANKS 16
36 /* 36 /*
37 * Each bank handles 32 irqs. Only the 16th (= last) bank handles only 37 * Each bank handles 32 irqs. Only the 16th (= last) bank handles only
38 * 16 irqs. 38 * 16 irqs.
39 */ 39 */
40 #define NVIC_MAX_IRQ ((NVIC_MAX_BANKS - 1) * 32 + 16) 40 #define NVIC_MAX_IRQ ((NVIC_MAX_BANKS - 1) * 32 + 16)
41 41
42 static struct irq_domain *nvic_irq_domain; 42 static struct irq_domain *nvic_irq_domain;
43 43
44 asmlinkage void __exception_irq_entry 44 asmlinkage void __exception_irq_entry
45 nvic_handle_irq(irq_hw_number_t hwirq, struct pt_regs *regs) 45 nvic_handle_irq(irq_hw_number_t hwirq, struct pt_regs *regs)
46 { 46 {
47 unsigned int irq = irq_linear_revmap(nvic_irq_domain, hwirq); 47 unsigned int irq = irq_linear_revmap(nvic_irq_domain, hwirq);
48 48
49 handle_IRQ(irq, regs); 49 handle_IRQ(irq, regs);
50 } 50 }
51 51
52 static void nvic_eoi(struct irq_data *d) 52 static void nvic_eoi(struct irq_data *d)
53 { 53 {
54 /* 54 /*
55 * This is a no-op as end of interrupt is signaled by the exception 55 * This is a no-op as end of interrupt is signaled by the exception
56 * return sequence. 56 * return sequence.
57 */ 57 */
58 } 58 }
59 59
60 static int __init nvic_of_init(struct device_node *node, 60 static int __init nvic_of_init(struct device_node *node,
61 struct device_node *parent) 61 struct device_node *parent)
62 { 62 {
63 unsigned int clr = IRQ_NOREQUEST | IRQ_NOPROBE | IRQ_NOAUTOEN; 63 unsigned int clr = IRQ_NOREQUEST | IRQ_NOPROBE | IRQ_NOAUTOEN;
64 unsigned int irqs, i, ret, numbanks; 64 unsigned int irqs, i, ret, numbanks;
65 void __iomem *nvic_base; 65 void __iomem *nvic_base;
66 66
67 numbanks = (readl_relaxed(V7M_SCS_ICTR) & 67 numbanks = (readl_relaxed(V7M_SCS_ICTR) &
68 V7M_SCS_ICTR_INTLINESNUM_MASK) + 1; 68 V7M_SCS_ICTR_INTLINESNUM_MASK) + 1;
69 69
70 nvic_base = of_iomap(node, 0); 70 nvic_base = of_iomap(node, 0);
71 if (!nvic_base) { 71 if (!nvic_base) {
72 pr_warn("unable to map nvic registers\n"); 72 pr_warn("unable to map nvic registers\n");
73 return -ENOMEM; 73 return -ENOMEM;
74 } 74 }
75 75
76 irqs = numbanks * 32; 76 irqs = numbanks * 32;
77 if (irqs > NVIC_MAX_IRQ) 77 if (irqs > NVIC_MAX_IRQ)
78 irqs = NVIC_MAX_IRQ; 78 irqs = NVIC_MAX_IRQ;
79 79
80 nvic_irq_domain = 80 nvic_irq_domain =
81 irq_domain_add_linear(node, irqs, &irq_generic_chip_ops, NULL); 81 irq_domain_add_linear(node, irqs, &irq_generic_chip_ops, NULL);
82 if (!nvic_irq_domain) { 82 if (!nvic_irq_domain) {
83 pr_warn("Failed to allocate irq domain\n"); 83 pr_warn("Failed to allocate irq domain\n");
84 return -ENOMEM; 84 return -ENOMEM;
85 } 85 }
86 86
87 ret = irq_alloc_domain_generic_chips(nvic_irq_domain, 32, numbanks, 87 ret = irq_alloc_domain_generic_chips(nvic_irq_domain, 32, 1,
88 "nvic_irq", handle_fasteoi_irq, 88 "nvic_irq", handle_fasteoi_irq,
89 clr, 0, IRQ_GC_INIT_MASK_CACHE); 89 clr, 0, IRQ_GC_INIT_MASK_CACHE);
90 if (ret) { 90 if (ret) {
91 pr_warn("Failed to allocate irq chips\n"); 91 pr_warn("Failed to allocate irq chips\n");
92 irq_domain_remove(nvic_irq_domain); 92 irq_domain_remove(nvic_irq_domain);
93 return ret; 93 return ret;
94 } 94 }
95 95
96 for (i = 0; i < numbanks; ++i) { 96 for (i = 0; i < numbanks; ++i) {
97 struct irq_chip_generic *gc; 97 struct irq_chip_generic *gc;
98 98
99 gc = irq_get_domain_generic_chip(nvic_irq_domain, 32 * i); 99 gc = irq_get_domain_generic_chip(nvic_irq_domain, 32 * i);
100 gc->reg_base = nvic_base + 4 * i; 100 gc->reg_base = nvic_base + 4 * i;
101 gc->chip_types[0].regs.enable = NVIC_ISER; 101 gc->chip_types[0].regs.enable = NVIC_ISER;
102 gc->chip_types[0].regs.disable = NVIC_ICER; 102 gc->chip_types[0].regs.disable = NVIC_ICER;
103 gc->chip_types[0].chip.irq_mask = irq_gc_mask_disable_reg; 103 gc->chip_types[0].chip.irq_mask = irq_gc_mask_disable_reg;
104 gc->chip_types[0].chip.irq_unmask = irq_gc_unmask_enable_reg; 104 gc->chip_types[0].chip.irq_unmask = irq_gc_unmask_enable_reg;
105 gc->chip_types[0].chip.irq_eoi = nvic_eoi; 105 gc->chip_types[0].chip.irq_eoi = nvic_eoi;
106 106
107 /* disable interrupts */ 107 /* disable interrupts */
108 writel_relaxed(~0, gc->reg_base + NVIC_ICER); 108 writel_relaxed(~0, gc->reg_base + NVIC_ICER);
109 } 109 }
110 110
111 /* Set priority on all interrupts */ 111 /* Set priority on all interrupts */
112 for (i = 0; i < irqs; i += 4) 112 for (i = 0; i < irqs; i += 4)
113 writel_relaxed(0, nvic_base + NVIC_IPR + i); 113 writel_relaxed(0, nvic_base + NVIC_IPR + i);
114 114
115 return 0; 115 return 0;
116 } 116 }
117 IRQCHIP_DECLARE(armv7m_nvic, "arm,armv7m-nvic", nvic_of_init); 117 IRQCHIP_DECLARE(armv7m_nvic, "arm,armv7m-nvic", nvic_of_init);
118 118