]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
sh: rework intc2 code
authorMagnus Damm <damm@igel.co.jp>
Fri, 15 Jun 2007 01:41:54 +0000 (10:41 +0900)
committerPaul Mundt <lethal@linux-sh.org>
Fri, 15 Jun 2007 01:41:54 +0000 (10:41 +0900)
The shared intc2 code currently contains cpu-specific #ifdefs.
This is a tad unclean and it prevents us from using the shared code
to drive board-specific irqs on the se7780 board.

This patch reworks the intc2 code by moving the base addresses of
the intc2 registers into struct intc2_desc. This new structure also
contains the name of the controller in struct irq_chip. The idea
behind putting struct irq_chip in there is that we can use offsetof()
to locate the base addresses in the irq_chip callbacks.

One logic change has been made - the original shared intc2 code
enabled the interrupts by default but with this patch they are all
disabled by default.

Signed-off-by: Magnus Damm <damm@igel.co.jp>
Signed-off-by: Paul Mundt <lethal@linux-sh.org>
arch/sh/boards/se/7780/irq.c
arch/sh/kernel/cpu/irq/intc2.c
arch/sh/kernel/cpu/sh4/setup-sh7760.c
arch/sh/kernel/cpu/sh4a/setup-sh7780.c
arch/sh/kernel/cpu/sh4a/setup-sh7785.c
include/asm-sh/hw_irq.h
include/asm-sh/irq.h

index 3d0625c2d07bd8106a39ee06606af7d0e28877e4..87491474600907cec7ced860f9ac5a7f284ca68c 100644 (file)
 #include <asm/io.h>
 #include <asm/se7780.h>
 
-#define INTC_INTMSK0             0xFFD00044
-#define INTC_INTMSKCLR0          0xFFD00064
-
-static void disable_se7780_irq(unsigned int irq)
-{
-       struct intc2_data *p = get_irq_chip_data(irq);
-       ctrl_outl(1 << p->msk_shift, INTC_INTMSK0 + p->msk_offset);
-}
-
-static void enable_se7780_irq(unsigned int irq)
-{
-       struct intc2_data *p = get_irq_chip_data(irq);
-       ctrl_outl(1 << p->msk_shift, INTC_INTMSKCLR0 + p->msk_offset);
-}
-
-static struct irq_chip se7780_irq_chip __read_mostly = {
-       .name           = "SE7780",
-       .mask           = disable_se7780_irq,
-       .unmask         = enable_se7780_irq,
-       .mask_ack       = disable_se7780_irq,
-};
-
 static struct intc2_data intc2_irq_table[] = {
        { 2,  0, 31, 0, 31, 3 }, /* daughter board EXTINT1 */
        { 4,  0, 30, 0, 30, 3 }, /* daughter board EXTINT2 */
@@ -51,13 +29,24 @@ static struct intc2_data intc2_irq_table[] = {
        { 0 , 0, 24, 0, 24, 3 }, /* SM501 */
 };
 
+static struct intc2_desc intc2_irq_desc __read_mostly = {
+       .prio_base      = 0, /* N/A */
+       .msk_base       = 0xffd00044,
+       .mskclr_base    = 0xffd00064,
+
+       .intc2_data     = intc2_irq_table,
+       .nr_irqs        = ARRAY_SIZE(intc2_irq_table),
+
+       .chip = {
+               .name   = "INTC2-se7780",
+       },
+};
+
 /*
  * Initialize IRQ setting
  */
 void __init init_se7780_IRQ(void)
 {
-       int i ;
-
        /* enable all interrupt at FPGA */
        ctrl_outw(0, FPGA_INTMSK1);
        /* mask SM501 interrupt */
@@ -79,11 +68,5 @@ void __init init_se7780_IRQ(void)
        /* FPGA + 0x0A */
        ctrl_outw((IRQPIN_PCCPW << IRQPOS_PCCPW), FPGA_INTSEL3);
 
-       for (i = 0; i < ARRAY_SIZE(intc2_irq_table); i++) {
-               disable_irq_nosync(intc2_irq_table[i].irq);
-               set_irq_chip_and_handler_name( intc2_irq_table[i].irq, &se7780_irq_chip,
-                       handle_level_irq, "level");
-               set_irq_chip_data( intc2_irq_table[i].irq, &intc2_irq_table[i] );
-               disable_se7780_irq(intc2_irq_table[i].irq);
-       }
+       register_intc2_controller(&intc2_irq_desc);
 }
index d8e22f4ff0f0a517497307fa45f4f1ff5322c4cf..dae02d3152e7ce45473b410c4a275ae565a77225 100644 (file)
 #include <linux/interrupt.h>
 #include <linux/io.h>
 
-#if defined(CONFIG_CPU_SUBTYPE_SH7760)
-#define INTC2_BASE     0xfe080000
-#define INTC2_INTMSK   (INTC2_BASE + 0x40)
-#define INTC2_INTMSKCLR        (INTC2_BASE + 0x60)
-#elif defined(CONFIG_CPU_SUBTYPE_SH7780) || \
-      defined(CONFIG_CPU_SUBTYPE_SH7785)
-#define INTC2_BASE     0xffd40000
-#define INTC2_INTMSK   (INTC2_BASE + 0x38)
-#define INTC2_INTMSKCLR        (INTC2_BASE + 0x3c)
-#endif
+static inline struct intc2_desc *get_intc2_desc(unsigned int irq)
+{
+       struct irq_chip *chip = get_irq_chip(irq);
+       return (void *)((char *)chip - offsetof(struct intc2_desc, chip));
+}
 
 static void disable_intc2_irq(unsigned int irq)
 {
        struct intc2_data *p = get_irq_chip_data(irq);
-       ctrl_outl(1 << p->msk_shift, INTC2_INTMSK + p->msk_offset);
+       struct intc2_desc *d = get_intc2_desc(irq);
+       ctrl_outl(1 << p->msk_shift, d->msk_base + p->msk_offset);
 }
 
 static void enable_intc2_irq(unsigned int irq)
 {
        struct intc2_data *p = get_irq_chip_data(irq);
-       ctrl_outl(1 << p->msk_shift, INTC2_INTMSKCLR + p->msk_offset);
+       struct intc2_desc *d = get_intc2_desc(irq);
+       ctrl_outl(1 << p->msk_shift, d->mskclr_base + p->msk_offset);
 }
 
-static struct irq_chip intc2_irq_chip = {
-       .name           = "INTC2",
-       .mask           = disable_intc2_irq,
-       .unmask         = enable_intc2_irq,
-       .mask_ack       = disable_intc2_irq,
-};
-
 /*
  * Setup an INTC2 style interrupt.
  * NOTE: Unlike IPR interrupts, parameters are not shifted by this code,
@@ -56,30 +46,36 @@ static struct irq_chip intc2_irq_chip = {
  *
  * in the intc2_data table.
  */
-void make_intc2_irq(struct intc2_data *table, unsigned int nr_irqs)
+void register_intc2_controller(struct intc2_desc *desc)
 {
        int i;
 
-       for (i = 0; i < nr_irqs; i++) {
+       desc->chip.mask = disable_intc2_irq;
+       desc->chip.unmask = enable_intc2_irq;
+       desc->chip.mask_ack = disable_intc2_irq;
+
+       for (i = 0; i < desc->nr_irqs; i++) {
                unsigned long ipr, flags;
-               struct intc2_data *p = table + i;
+               struct intc2_data *p = desc->intc2_data + i;
 
                disable_irq_nosync(p->irq);
 
-               /* Set the priority level */
-               local_irq_save(flags);
+               if (desc->prio_base) {
+                       /* Set the priority level */
+                       local_irq_save(flags);
 
-               ipr = ctrl_inl(INTC2_BASE + p->ipr_offset);
-               ipr &= ~(0xf << p->ipr_shift);
-               ipr |= p->priority << p->ipr_shift;
-               ctrl_outl(ipr, INTC2_BASE + p->ipr_offset);
+                       ipr = ctrl_inl(desc->prio_base + p->ipr_offset);
+                       ipr &= ~(0xf << p->ipr_shift);
+                       ipr |= p->priority << p->ipr_shift;
+                       ctrl_outl(ipr, desc->prio_base + p->ipr_offset);
 
-               local_irq_restore(flags);
+                       local_irq_restore(flags);
+               }
 
-               set_irq_chip_and_handler_name(p->irq, &intc2_irq_chip,
+               set_irq_chip_and_handler_name(p->irq, &desc->chip,
                                              handle_level_irq, "level");
                set_irq_chip_data(p->irq, p);
 
-               enable_intc2_irq(p->irq);
+               disable_intc2_irq(p->irq);
        }
 }
index b7c702821e6fed19ef060e0e2469ec7619246c71..6d3c91897774221345dc1c04ccff118285867df3 100644 (file)
@@ -96,6 +96,19 @@ static struct intc2_data intc2_irq_table[] = {
        {109,12,  0, 4,  0,  3},        /* CMTI */
 };
 
+static struct intc2_desc intc2_irq_desc __read_mostly = {
+       .prio_base      = 0xfe080000,
+       .msk_base       = 0xfe080040,
+       .mskclr_base    = 0xfe080060,
+
+       .intc2_data     = intc2_irq_table,
+       .nr_irqs        = ARRAY_SIZE(intc2_irq_table),
+
+       .chip = {
+               .name   = "INTC2-sh7760",
+       },
+};
+
 static struct ipr_data sh7760_ipr_map[] = {
        /* IRQ, IPR-idx, shift, priority */
        { 16, 0, 12, 2 }, /* TMU0 TUNI*/
@@ -143,7 +156,7 @@ unsigned int map_ipridx_to_addr(int idx)
 
 void __init init_IRQ_intc2(void)
 {
-       make_intc2_irq(intc2_irq_table, ARRAY_SIZE(intc2_irq_table));
+       register_intc2_controller(&intc2_irq_desc);
 }
 
 void __init  init_IRQ_ipr(void)
index 9aeaa2ddaa28d38a0e4e39f8ca6030adf4259671..b57c760bffde4abdc6af3505a864eda18f4a4f2f 100644 (file)
@@ -102,7 +102,20 @@ static struct intc2_data intc2_irq_table[] = {
        { 68, 0x14,  8, 0, 18, 2 },     /* PCIC4 */
 };
 
+static struct intc2_desc intc2_irq_desc __read_mostly = {
+       .prio_base      = 0xffd40000,
+       .msk_base       = 0xffd40038,
+       .mskclr_base    = 0xffd4003c,
+
+       .intc2_data     = intc2_irq_table,
+       .nr_irqs        = ARRAY_SIZE(intc2_irq_table),
+
+       .chip = {
+               .name   = "INTC2-sh7780",
+       },
+};
+
 void __init init_IRQ_intc2(void)
 {
-       make_intc2_irq(intc2_irq_table, ARRAY_SIZE(intc2_irq_table));
+       register_intc2_controller(&intc2_irq_desc);
 }
index 07b0de82cfe6b280e1848265a0ace7f13aab0bba..ce10ec5d6914dcd55e2fecc608338d5968e57820 100644 (file)
@@ -97,7 +97,21 @@ static struct intc2_data intc2_irq_table[] = {
        { 60, 12, 16, 0, 7, 3 },        /* SCIF5 ERI, RXI, BRI, TXI */
 };
 
+static struct intc2_desc intc2_irq_desc __read_mostly = {
+       .prio_base      = 0xffd40000,
+       .msk_base       = 0xffd40038,
+       .mskclr_base    = 0xffd4003c,
+
+       .intc2_data     = intc2_irq_table,
+       .nr_irqs        = ARRAY_SIZE(intc2_irq_table),
+
+       .chip = {
+               .name   = "INTC2-sh7785",
+       },
+};
+
 void __init init_IRQ_intc2(void)
 {
-       make_intc2_irq(intc2_irq_table, ARRAY_SIZE(intc2_irq_table));
+       register_intc2_controller(&intc2_irq_desc);
 }
+
index 80ee1cda7498f2383cd72588a800c66ad68178e3..f9dfdc04aef50607475c6e43e68b3180c6700dda 100644 (file)
@@ -5,4 +5,23 @@
 
 extern atomic_t irq_err_count;
 
+struct intc2_data {
+       unsigned short irq;
+       unsigned char ipr_offset, ipr_shift;
+       unsigned char msk_offset, msk_shift;
+       unsigned char priority;
+};
+
+struct intc2_desc {
+       unsigned long prio_base;
+       unsigned long msk_base;
+       unsigned long mskclr_base;
+       struct intc2_data *intc2_data;
+       unsigned int nr_irqs;
+       struct irq_chip chip;
+};
+
+void register_intc2_controller(struct intc2_desc *);
+void init_IRQ_intc2(void);
+
 #endif /* __ASM_SH_HW_IRQ_H */
index e81bf21c801e77f7ac03ef19f9058ce101cfd8e1..c247b733a7d6f6a7622e7467f3398495b283a701 100644 (file)
@@ -63,16 +63,6 @@ void make_ipr_irq(struct ipr_data *table, unsigned int nr_irqs);
 void make_imask_irq(unsigned int irq);
 void init_IRQ_ipr(void);
 
-struct intc2_data {
-       unsigned short irq;
-       unsigned char ipr_offset, ipr_shift;
-       unsigned char msk_offset, msk_shift;
-       unsigned char priority;
-};
-
-void make_intc2_irq(struct intc2_data *, unsigned int nr_irqs);
-void init_IRQ_intc2(void);
-
 static inline int generic_irq_demux(int irq)
 {
        return irq;