Commit af80b0fe authored by Gerlando Falauto's avatar Gerlando Falauto Committed by Thomas Gleixner

genirq: Generic chip: Handle separate mask registers

There are cases where all irq_chip_type instances have separate mask
registers, making a shared mask register cache unsuitable for the
purpose.

Introduce a new flag IRQ_GC_MASK_CACHE_PER_TYPE. If set, point the per
chip mask pointer to the per chip private mask cache instead.

[ tglx: Simplified code, renamed flag and massaged changelog ]
Signed-off-by: default avatarGerlando Falauto <gerlando.falauto@keymile.com>
Cc: Andrew Lunn <andrew@lunn.ch>
Cc: Joey Oravec <joravec@drewtech.com>
Cc: Lennert Buytenhek <kernel@wantstofly.org>
Cc: Russell King - ARM Linux <linux@arm.linux.org.uk>
Cc: Jason Gunthorpe <jgunthorpe@obsidianresearch.com>
Cc: Holger Brunck <Holger.Brunck@keymile.com>
Cc: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
Acked-by: default avatarGrant Likely <grant.likely@linaro.org>
Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
Cc: Jason Cooper <jason@lakedaemon.net>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: devicetree-discuss@lists.ozlabs.org
Cc: Rob Herring <rob.herring@calxeda.com>
Cc: Ben Dooks <ben-linux@fluff.org>
Cc: Gregory Clement <gregory.clement@free-electrons.com>
Cc: Simon Guinot <simon@sequanux.org>
Cc: linux-arm-kernel@lists.infradead.org
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Cc: Jean-Francois Moine <moinejf@free.fr>
Cc: Nicolas Pitre <nico@fluxnic.net>
Cc: Rob Landley <rob@landley.net>
Cc: Maxime Ripard <maxime.ripard@free-electrons.com>
Link: http://lkml.kernel.org/r/20130506142539.152569748@linutronix.deSigned-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
parent 899f0e66
...@@ -704,10 +704,12 @@ struct irq_chip_generic { ...@@ -704,10 +704,12 @@ struct irq_chip_generic {
* @IRQ_GC_INIT_NESTED_LOCK: Set the lock class of the irqs to nested for * @IRQ_GC_INIT_NESTED_LOCK: Set the lock class of the irqs to nested for
* irq chips which need to call irq_set_wake() on * irq chips which need to call irq_set_wake() on
* the parent irq. Usually GPIO implementations * the parent irq. Usually GPIO implementations
* @IRQ_GC_MASK_CACHE_PER_TYPE: Mask cache is chip type private
*/ */
enum irq_gc_flags { enum irq_gc_flags {
IRQ_GC_INIT_MASK_CACHE = 1 << 0, IRQ_GC_INIT_MASK_CACHE = 1 << 0,
IRQ_GC_INIT_NESTED_LOCK = 1 << 1, IRQ_GC_INIT_NESTED_LOCK = 1 << 1,
IRQ_GC_MASK_CACHE_PER_TYPE = 1 << 2,
}; };
/* Generic chip callback functions */ /* Generic chip callback functions */
......
...@@ -241,18 +241,21 @@ void irq_setup_generic_chip(struct irq_chip_generic *gc, u32 msk, ...@@ -241,18 +241,21 @@ void irq_setup_generic_chip(struct irq_chip_generic *gc, u32 msk,
{ {
struct irq_chip_type *ct = gc->chip_types; struct irq_chip_type *ct = gc->chip_types;
unsigned int i; unsigned int i;
u32 *mskptr = &gc->mask_cache, mskreg = ct->regs.mask;
raw_spin_lock(&gc_lock); raw_spin_lock(&gc_lock);
list_add_tail(&gc->list, &gc_list); list_add_tail(&gc->list, &gc_list);
raw_spin_unlock(&gc_lock); raw_spin_unlock(&gc_lock);
/* Init mask cache ? */ for (i = 0; i < gc->num_ct; i++) {
if (flags & IRQ_GC_INIT_MASK_CACHE) if (flags & IRQ_GC_MASK_CACHE_PER_TYPE) {
gc->mask_cache = irq_reg_readl(gc->reg_base + ct->regs.mask); mskptr = &ct[i].mask_cache_priv;
mskreg = ct[i].regs.mask;
/* Initialize mask cache pointer */ }
for (i = 0; i < gc->num_ct; i++) ct[i].mask_cache = mskptr;
ct[i].mask_cache = &gc->mask_cache; if (flags & IRQ_GC_INIT_MASK_CACHE)
*mskptr = irq_reg_readl(gc->reg_base + mskreg);
}
for (i = gc->irq_base; msk; msk >>= 1, i++) { for (i = gc->irq_base; msk; msk >>= 1, i++) {
if (!(msk & 0x01)) if (!(msk & 0x01))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment