* [PATCH 0/2] arm: samsung: add device tree support for gic and interrupt combiner @ 2011-10-06 21:11 Thomas Abraham 2011-10-06 21:11 ` [PATCH 1/2] arm: samsung: move timer irq numbers to end of linux irq space Thomas Abraham 2011-10-06 21:11 ` [PATCH 2/2] arm: exynos4: add support for dt irq specifier to linux virq conversion Thomas Abraham 0 siblings, 2 replies; 6+ messages in thread From: Thomas Abraham @ 2011-10-06 21:11 UTC (permalink / raw) To: linux-arm-kernel This patchset adds device tree support for GIC and Interrupt Combiner controllers in the Exynos4 SoC. Patch 1 moves the statically mapped timer irqs 11 to 15 to the end of the statically mapped linux irq space for all s5p platforms. This was required because, device tree support for gic is based on Rob Herring's "GIC OF bindings" patchset and that patchset requires hardware irq number to be equal to GIC interrupt ID. Since GIC interrupt ID starts from zero, the timer irqs 11 to 15 is moved to the end of the irq space and hardware irqs are statically mapped to start from zero. I am not sure if statically mapping hardware irqs starting from zero would be a problem. Patch 2 adds device tree support for GIC and Interrupt Combiner controllers on Exynos4. For GIC controller, this patch is based on Rob Herring's, "[PATCH 0/3] GIC OF bindings" patchset. The concept of irq domain for interrupt combiner controller is based on Grant's 'simple' irq converter Thomas Abraham (2): arm: samsung: move timer irq numbers to end of linux irq space arm: exynos4: add support for dt irq specifier to linux virq conversion arch/arm/mach-exynos4/cpu.c | 20 +++++++++- arch/arm/mach-exynos4/include/mach/entry-macro.S | 1 - arch/arm/mach-exynos4/include/mach/irqs.h | 3 +- arch/arm/mach-exynos4/irq-combiner.c | 44 ++++++++++++++++++++++ arch/arm/mach-s5p64x0/include/mach/irqs.h | 4 +- arch/arm/mach-s5pc100/include/mach/irqs.h | 3 +- arch/arm/mach-s5pv210/include/mach/irqs.h | 3 +- arch/arm/plat-samsung/include/plat/irqs.h | 7 ++- 8 files changed, 72 insertions(+), 13 deletions(-) ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/2] arm: samsung: move timer irq numbers to end of linux irq space 2011-10-06 21:11 [PATCH 0/2] arm: samsung: add device tree support for gic and interrupt combiner Thomas Abraham @ 2011-10-06 21:11 ` Thomas Abraham 2011-10-06 22:15 ` Russell King - ARM Linux 2011-10-06 21:11 ` [PATCH 2/2] arm: exynos4: add support for dt irq specifier to linux virq conversion Thomas Abraham 1 sibling, 1 reply; 6+ messages in thread From: Thomas Abraham @ 2011-10-06 21:11 UTC (permalink / raw) To: linux-arm-kernel All of Samsung's s5p platforms have timer irqs statically mapped from linux irq numbers 11 to 15. These timer irqs are moved to end of the statically mapped linux irq space and the hardware irqs, which were statically mapped starting from 32 is moved to start from 0. The NR_IRQS macro is consolidated for all the s5p platforms in this process. Cc: Ben Dooks <ben-linux@fluff.org> Cc: Kukjin Kim <kgene.kim@samsung.com> Signed-off-by: Thomas Abraham <thomas.abraham@linaro.org> --- arch/arm/mach-exynos4/include/mach/entry-macro.S | 1 - arch/arm/mach-exynos4/include/mach/irqs.h | 3 +-- arch/arm/mach-s5p64x0/include/mach/irqs.h | 4 +--- arch/arm/mach-s5pc100/include/mach/irqs.h | 3 +-- arch/arm/mach-s5pv210/include/mach/irqs.h | 3 +-- arch/arm/plat-samsung/include/plat/irqs.h | 7 +++++-- 6 files changed, 9 insertions(+), 12 deletions(-) diff --git a/arch/arm/mach-exynos4/include/mach/entry-macro.S b/arch/arm/mach-exynos4/include/mach/entry-macro.S index 4c9adbd..5c4fbcc 100644 --- a/arch/arm/mach-exynos4/include/mach/entry-macro.S +++ b/arch/arm/mach-exynos4/include/mach/entry-macro.S @@ -72,7 +72,6 @@ cmpcc \irqnr, \irqnr cmpne \irqnr, \tmp cmpcs \irqnr, \irqnr - addne \irqnr, \irqnr, #32 .endm diff --git a/arch/arm/mach-exynos4/include/mach/irqs.h b/arch/arm/mach-exynos4/include/mach/irqs.h index dfd4b7e..43087c3 100644 --- a/arch/arm/mach-exynos4/include/mach/irqs.h +++ b/arch/arm/mach-exynos4/include/mach/irqs.h @@ -163,7 +163,6 @@ #define IRQ_GPIO2_NR_GROUPS 9 #define IRQ_GPIO_END (S5P_GPIOINT_BASE + S5P_GPIOINT_COUNT) -/* Set the default NR_IRQS */ -#define NR_IRQS (IRQ_GPIO_END + 64) +#define IRQ_TIMER_BASE (IRQ_GPIO_END + 64) #endif /* __ASM_ARCH_IRQS_H */ diff --git a/arch/arm/mach-s5p64x0/include/mach/irqs.h b/arch/arm/mach-s5p64x0/include/mach/irqs.h index 53982db..bea73cc 100644 --- a/arch/arm/mach-s5p64x0/include/mach/irqs.h +++ b/arch/arm/mach-s5p64x0/include/mach/irqs.h @@ -141,8 +141,6 @@ #define IRQ_EINT_GROUP(grp, x) (IRQ_EINT_GROUP##grp##_BASE + (x)) -/* Set the default NR_IRQS */ - -#define NR_IRQS (IRQ_EINT_GROUP8_BASE + IRQ_EINT_GROUP8_NR + 1) +#define IRQ_TIMER_BASE (IRQ_EINT_GROUP8_BASE + IRQ_EINT_GROUP8_NR + 1) #endif /* __ASM_ARCH_IRQS_H */ diff --git a/arch/arm/mach-s5pc100/include/mach/irqs.h b/arch/arm/mach-s5pc100/include/mach/irqs.h index d2eb475..3a9d300 100644 --- a/arch/arm/mach-s5pc100/include/mach/irqs.h +++ b/arch/arm/mach-s5pc100/include/mach/irqs.h @@ -104,8 +104,7 @@ #define S5P_GPIOINT_BASE (IRQ_EINT(31) + 1) #define S5P_GPIOINT_GROUP_MAXNR 21 -/* Set the default NR_IRQS */ -#define NR_IRQS (IRQ_EINT(31) + S5P_GPIOINT_COUNT + 1) +#define IRQ_TIMER_BASE (IRQ_EINT(31) + S5P_GPIOINT_COUNT + 1) /* Compatibility */ #define IRQ_LCD_FIFO IRQ_LCD0 diff --git a/arch/arm/mach-s5pv210/include/mach/irqs.h b/arch/arm/mach-s5pv210/include/mach/irqs.h index 5e0de3a..df3173a 100644 --- a/arch/arm/mach-s5pv210/include/mach/irqs.h +++ b/arch/arm/mach-s5pv210/include/mach/irqs.h @@ -125,8 +125,7 @@ #define S5P_GPIOINT_BASE (IRQ_EINT(31) + 1) #define S5P_GPIOINT_GROUP_MAXNR 22 -/* Set the default NR_IRQS */ -#define NR_IRQS (IRQ_EINT(31) + S5P_GPIOINT_COUNT + 1) +#define IRQ_TIMER_BASE (IRQ_EINT(31) + S5P_GPIOINT_COUNT + 1) /* Compatibility */ #define IRQ_LCD_FIFO IRQ_LCD0 diff --git a/arch/arm/plat-samsung/include/plat/irqs.h b/arch/arm/plat-samsung/include/plat/irqs.h index 08d1a7e..b8918b3 100644 --- a/arch/arm/plat-samsung/include/plat/irqs.h +++ b/arch/arm/plat-samsung/include/plat/irqs.h @@ -22,7 +22,7 @@ * mulitple of 32 to allow the common code to work */ -#define S5P_IRQ_OFFSET (32) +#define S5P_IRQ_OFFSET (0) #define S5P_IRQ(x) ((x) + S5P_IRQ_OFFSET) @@ -44,13 +44,14 @@ #define S5P_IRQ_VIC2(x) (S5P_VIC2_BASE + (x)) #define S5P_IRQ_VIC3(x) (S5P_VIC3_BASE + (x)) -#define S5P_TIMER_IRQ(x) (11 + (x)) +#define S5P_TIMER_IRQ(x) (IRQ_TIMER_BASE + (x)) #define IRQ_TIMER0 S5P_TIMER_IRQ(0) #define IRQ_TIMER1 S5P_TIMER_IRQ(1) #define IRQ_TIMER2 S5P_TIMER_IRQ(2) #define IRQ_TIMER3 S5P_TIMER_IRQ(3) #define IRQ_TIMER4 S5P_TIMER_IRQ(4) +#define IRQ_TIMER_COUNT (5) #define IRQ_EINT(x) ((x) < 16 ? ((x) + S5P_EINT_BASE1) \ : ((x) - 16 + S5P_EINT_BASE2)) @@ -77,4 +78,6 @@ #define S5P_IRQ_TYPE_EDGE_RISING (0x03) #define S5P_IRQ_TYPE_EDGE_BOTH (0x04) +#define NR_IRQS (IRQ_TIMER_BASE + IRQ_TIMER_COUNT) + #endif /* __PLAT_SAMSUNG_IRQS_H */ -- 1.6.6.rc2 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 1/2] arm: samsung: move timer irq numbers to end of linux irq space 2011-10-06 21:11 ` [PATCH 1/2] arm: samsung: move timer irq numbers to end of linux irq space Thomas Abraham @ 2011-10-06 22:15 ` Russell King - ARM Linux 2011-10-06 23:56 ` Thomas Abraham 0 siblings, 1 reply; 6+ messages in thread From: Russell King - ARM Linux @ 2011-10-06 22:15 UTC (permalink / raw) To: linux-arm-kernel On Fri, Oct 07, 2011 at 02:41:42AM +0530, Thomas Abraham wrote: > All of Samsung's s5p platforms have timer irqs statically mapped from linux > irq numbers 11 to 15. These timer irqs are moved to end of the statically > mapped linux irq space and the hardware irqs, which were statically mapped > starting from 32 is moved to start from 0. The NR_IRQS macro is consolidated > for all the s5p platforms in this process. Am I reading this patch correctly - in that on platforms with the GIC, you add 32 to the GIC IRQ number _just_ to avoid the possibility that IRQs 11-15 are seen as valid? Or to put it another way, you're trying to ensure that if these timer IRQs are requested, they will fail? ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/2] arm: samsung: move timer irq numbers to end of linux irq space 2011-10-06 22:15 ` Russell King - ARM Linux @ 2011-10-06 23:56 ` Thomas Abraham 2011-10-06 23:59 ` Thomas Abraham 0 siblings, 1 reply; 6+ messages in thread From: Thomas Abraham @ 2011-10-06 23:56 UTC (permalink / raw) To: linux-arm-kernel Hi Russell, On 7 October 2011 03:45, Russell King - ARM Linux <linux@arm.linux.org.uk> wrote: > On Fri, Oct 07, 2011 at 02:41:42AM +0530, Thomas Abraham wrote: >> All of Samsung's s5p platforms have timer irqs statically mapped from linux >> irq numbers 11 to 15. These timer irqs are moved to end of the statically >> mapped linux irq space and the hardware irqs, which were statically mapped >> starting from 32 is moved to start from 0. The NR_IRQS macro is consolidated >> for all the s5p platforms in this process. > > Am I reading this patch correctly - in that on platforms with the GIC, > you add 32 to the GIC IRQ number _just_ to avoid the possibility that > IRQs 11-15 are seen as valid? > > Or to put it another way, you're trying to ensure that if these timer > IRQs are requested, they will fail? > This patch is applicable for s5p64x0, s5pc100, s5pv210 and exynos4. Exynos4 has GIC and the rest use VIC. For all these platforms, the five hardware timer irqs are connected to GIC/VIC at some hardware irq number (in exynos it is GIC_ID 69 to 73 for five timers). When any of these hardware interrupt occurs, its interrupt handler calls generic_handle_irq() with linux irq number 11/12/13/14/15 for timer 0/1/2/3/4 as the parameter. The code that needs to be notified about the timer interrupts would have already registered its handler for either of the interrupts 11 to 15. I am not sure why such remapping is used. I think it is not needed but did not intend to change it. All this is included in the file "arch/arm/plat-samsung/irq-vic-timer.c". Instead of using linux irq number 11 to 15 to which consumers of timer interrupt attach their handler, this interrupt range is moved to the end of linux irq space used. So there will be no interrupts statically mapped between 0 to 31. The GIC/VIC hardware interrupts, which were previously statically mapped to start from linux irq 32 are now moved to start from linux irq 0. In case of exynos, GIC_ID[0] (which is SGI[0]) which was previously at linux irq 32, will not be at linux irq 0. This was the intended change. This was required to use Rob Herring's GIC OF bindings patches for Exynos4. Thanks, Thomas. ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/2] arm: samsung: move timer irq numbers to end of linux irq space 2011-10-06 23:56 ` Thomas Abraham @ 2011-10-06 23:59 ` Thomas Abraham 0 siblings, 0 replies; 6+ messages in thread From: Thomas Abraham @ 2011-10-06 23:59 UTC (permalink / raw) To: linux-arm-kernel On 7 October 2011 05:26, Thomas Abraham <thomas.abraham@linaro.org> wrote: > Hi Russell, > > On 7 October 2011 03:45, Russell King - ARM Linux > <linux@arm.linux.org.uk> wrote: >> On Fri, Oct 07, 2011 at 02:41:42AM +0530, Thomas Abraham wrote: >>> All of Samsung's s5p platforms have timer irqs statically mapped from linux >>> irq numbers 11 to 15. These timer irqs are moved to end of the statically >>> mapped linux irq space and the hardware irqs, which were statically mapped >>> starting from 32 is moved to start from 0. The NR_IRQS macro is consolidated >>> for all the s5p platforms in this process. >> >> Am I reading this patch correctly - in that on platforms with the GIC, >> you add 32 to the GIC IRQ number _just_ to avoid the possibility that >> IRQs 11-15 are seen as valid? >> >> Or to put it another way, you're trying to ensure that if these timer >> IRQs are requested, they will fail? >> > > This patch is applicable for s5p64x0, s5pc100, s5pv210 and exynos4. > Exynos4 has GIC and the rest use VIC. > > For all these platforms, the five hardware timer irqs are connected to > GIC/VIC at some hardware irq number (in exynos it is GIC_ID 69 to 73 > for five timers). When any of these hardware interrupt occurs, its > interrupt handler calls generic_handle_irq() with linux irq number > 11/12/13/14/15 for timer 0/1/2/3/4 as the parameter. The code that > needs to be notified about the timer interrupts would have already > registered its handler for either of the interrupts 11 to 15. I am not > sure why such remapping is used. I think it is not needed but did not > intend to change it. All this is included in the file > "arch/arm/plat-samsung/irq-vic-timer.c". > > Instead of using linux irq number 11 to 15 to which consumers of timer > interrupt attach their handler, this interrupt range is moved to the > end of linux irq space used. So there will be no interrupts statically > mapped between 0 to 31. > > The GIC/VIC hardware interrupts, which were previously statically > mapped to start from linux irq 32 are now moved to start from linux > irq 0. In case of exynos, GIC_ID[0] (which is SGI[0]) which was > previously at linux irq 32, will not be at linux irq 0. Correcting the line above: previously at linux irq 32, will _now_ be at linux irq 0. > > This was the intended change. This was required to use Rob Herring's > GIC OF bindings patches for Exynos4. > > Thanks, > Thomas. > ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 2/2] arm: exynos4: add support for dt irq specifier to linux virq conversion 2011-10-06 21:11 [PATCH 0/2] arm: samsung: add device tree support for gic and interrupt combiner Thomas Abraham 2011-10-06 21:11 ` [PATCH 1/2] arm: samsung: move timer irq numbers to end of linux irq space Thomas Abraham @ 2011-10-06 21:11 ` Thomas Abraham 1 sibling, 0 replies; 6+ messages in thread From: Thomas Abraham @ 2011-10-06 21:11 UTC (permalink / raw) To: linux-arm-kernel Add support for conversion of device tree interrupt specifier to linux virq domain for GIC and Interrupt combiner controllers. Cc: Rob Herring <rob.herring@calxeda.com> Cc: Grant Likely <grant.likely@secretlab.ca> Signed-off-by: Thomas Abraham <thomas.abraham@linaro.org> --- arch/arm/mach-exynos4/cpu.c | 20 ++++++++++++++- arch/arm/mach-exynos4/irq-combiner.c | 44 ++++++++++++++++++++++++++++++++++ 2 files changed, 63 insertions(+), 1 deletions(-) diff --git a/arch/arm/mach-exynos4/cpu.c b/arch/arm/mach-exynos4/cpu.c index 1c5e803..9898f9c 100644 --- a/arch/arm/mach-exynos4/cpu.c +++ b/arch/arm/mach-exynos4/cpu.c @@ -10,6 +10,8 @@ #include <linux/sched.h> #include <linux/sysdev.h> +#include <linux/of.h> +#include <linux/of_irq.h> #include <asm/mach/map.h> #include <asm/mach/irq.h> @@ -38,6 +40,8 @@ unsigned int gic_bank_offset __read_mostly; extern int combiner_init(unsigned int combiner_nr, void __iomem *base, unsigned int irq_start); extern void combiner_cascade_irq(unsigned int combiner_nr, unsigned int irq); +extern void combiner_init_dt(struct device_node *node, + struct device_node *parent); /* Initial IO mappings */ static struct map_desc exynos4_iodesc[] __initdata = { @@ -218,13 +222,27 @@ static void exynos4_gic_irq_fix_base(struct irq_data *d) (gic_bank_offset * smp_processor_id()); } +#ifdef CONFIG_OF +static const struct of_device_id exynos4_dt_irq_match[] = { + { .compatible = "arm,cortex-a9-gic", .data = gic_of_init, }, + { .compatible = "samsung,exynos4-combiner", .data = combiner_init_dt, }, + {}, +}; +#endif + void __init exynos4_init_irq(void) { int irq; gic_bank_offset = soc_is_exynos4412() ? 0x4000 : 0x8000; - gic_init(0, IRQ_PPI(0), S5P_VA_GIC_DIST, S5P_VA_GIC_CPU); +#ifdef CONFIG_OF + if (of_have_populated_dt()) + of_irq_init(exynos4_dt_irq_match); + else +#endif + gic_init(0, IRQ_PPI(0), S5P_VA_GIC_DIST, S5P_VA_GIC_CPU); + gic_arch_extn.irq_eoi = exynos4_gic_irq_fix_base; gic_arch_extn.irq_unmask = exynos4_gic_irq_fix_base; gic_arch_extn.irq_mask = exynos4_gic_irq_fix_base; diff --git a/arch/arm/mach-exynos4/irq-combiner.c b/arch/arm/mach-exynos4/irq-combiner.c index 5a2758a..d2d143a 100644 --- a/arch/arm/mach-exynos4/irq-combiner.c +++ b/arch/arm/mach-exynos4/irq-combiner.c @@ -13,6 +13,10 @@ */ #include <linux/io.h> +#include <linux/of.h> +#include <linux/of_irq.h> +#include <linux/irqdomain.h> +#include <linux/slab.h> #include <asm/mach/irq.h> @@ -122,3 +126,43 @@ void __init combiner_init(unsigned int combiner_nr, void __iomem *base, set_irq_flags(i, IRQF_VALID | IRQF_PROBE); } } + +#ifdef CONFIG_OF +/* Translate dt irq specifier to linux virq for interrupt combiner controller */ +static int exynos4_irq_domain_combiner_dt_translate(struct irq_domain *d, + struct device_node *controller, + const u32 *intspec, unsigned int intsize, + unsigned long *out_hwirq, unsigned int *out_type) +{ + if (d->of_node != controller) + return -EINVAL; + if (intsize < 2) + return -EINVAL; + + *out_hwirq = COMBINER_IRQ(intspec[0], intspec[1]); + *out_type = IRQ_TYPE_NONE; + return 0; +} + +static struct irq_domain_ops exynos4_irq_domain_combiner_ops = { + .dt_translate = exynos4_irq_domain_combiner_dt_translate, +}; + +void __init combiner_init_dt(struct device_node *node, + struct device_node *parent) +{ + struct irq_domain *domain; + + if (WARN(!node, "combiner_init_dt: invalid node in parameter\n")) + return; + + domain = kzalloc(sizeof(*domain), GFP_KERNEL); + if (domain) { + domain->of_node = node; + domain->ops = &exynos4_irq_domain_combiner_ops; + irq_domain_add(domain); + } else { + WARN_ON(1); + } +} +#endif -- 1.6.6.rc2 ^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-10-06 23:59 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2011-10-06 21:11 [PATCH 0/2] arm: samsung: add device tree support for gic and interrupt combiner Thomas Abraham 2011-10-06 21:11 ` [PATCH 1/2] arm: samsung: move timer irq numbers to end of linux irq space Thomas Abraham 2011-10-06 22:15 ` Russell King - ARM Linux 2011-10-06 23:56 ` Thomas Abraham 2011-10-06 23:59 ` Thomas Abraham 2011-10-06 21:11 ` [PATCH 2/2] arm: exynos4: add support for dt irq specifier to linux virq conversion Thomas Abraham
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).