From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965355Ab3E2JSO (ORCPT ); Wed, 29 May 2013 05:18:14 -0400 Received: from terminus.zytor.com ([198.137.202.10]:59683 "EHLO terminus.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965261Ab3E2JSK (ORCPT ); Wed, 29 May 2013 05:18:10 -0400 Date: Wed, 29 May 2013 02:17:16 -0700 From: tip-bot for Gerlando Falauto Message-ID: Cc: mingo@kernel.org, moinejf@free.fr, jgunthorpe@obsidianresearch.com, arnd@arndb.de, Holger.Brunck@keymile.com, thomas.petazzoni@free-electrons.com, kernel@wantstofly.org, linux@arm.linux.org.uk, simon@sequanux.org, ezequiel.garcia@free-electrons.com, tglx@linutronix.de, joravec@drewtech.com, maxime.ripard@free-electrons.com, rob@landley.net, nico@fluxnic.net, linux-kernel@vger.kernel.org, ben-linux@fluff.org, hpa@zytor.com, grant.likely@linaro.org, jason@lakedaemon.net, gerlando.falauto@keymile.com, gregory.clement@free-electrons.com, sebastian.hesselbarth@gmail.com, rob.herring@calxeda.com, andrew@lunn.ch Reply-To: mingo@kernel.org, moinejf@free.fr, jgunthorpe@obsidianresearch.com, arnd@arndb.de, Holger.Brunck@keymile.com, thomas.petazzoni@free-electrons.com, kernel@wantstofly.org, linux@arm.linux.org.uk, joravec@drewtech.com, tglx@linutronix.de, ezequiel.garcia@free-electrons.com, simon@sequanux.org, maxime.ripard@free-electrons.com, nico@fluxnic.net, rob@landley.net, jason@lakedaemon.net, grant.likely@linaro.org, hpa@zytor.com, ben-linux@fluff.org, linux-kernel@vger.kernel.org, gerlando.falauto@keymile.com, rob.herring@calxeda.com, sebastian.hesselbarth@gmail.com, gregory.clement@free-electrons.com, andrew@lunn.ch In-Reply-To: <20130506142539.152569748@linutronix.de> References: <20130506142539.152569748@linutronix.de> To: linux-tip-commits@vger.kernel.org Subject: [tip:irq/core] genirq: Generic chip: Handle separate mask registers Git-Commit-ID: af80b0fed67261dcba2ce2406db1d553d07cbe75 X-Mailer: tip-git-log-daemon Robot-ID: Robot-Unsubscribe: Contact to get blacklisted from these emails MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Commit-ID: af80b0fed67261dcba2ce2406db1d553d07cbe75 Gitweb: http://git.kernel.org/tip/af80b0fed67261dcba2ce2406db1d553d07cbe75 Author: Gerlando Falauto AuthorDate: Mon, 6 May 2013 14:30:21 +0000 Committer: Thomas Gleixner CommitDate: Wed, 29 May 2013 10:57:10 +0200 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: Gerlando Falauto Cc: Andrew Lunn Cc: Joey Oravec Cc: Lennert Buytenhek Cc: Russell King - ARM Linux Cc: Jason Gunthorpe Cc: Holger Brunck Cc: Ezequiel Garcia Acked-by: Grant Likely Cc: Sebastian Hesselbarth Cc: Jason Cooper Cc: Arnd Bergmann Cc: devicetree-discuss@lists.ozlabs.org Cc: Rob Herring Cc: Ben Dooks Cc: Gregory Clement Cc: Simon Guinot Cc: linux-arm-kernel@lists.infradead.org Cc: Thomas Petazzoni Cc: Jean-Francois Moine Cc: Nicolas Pitre Cc: Rob Landley Cc: Maxime Ripard Link: http://lkml.kernel.org/r/20130506142539.152569748@linutronix.de Signed-off-by: Thomas Gleixner --- include/linux/irq.h | 2 ++ kernel/irq/generic-chip.c | 17 ++++++++++------- 2 files changed, 12 insertions(+), 7 deletions(-) diff --git a/include/linux/irq.h b/include/linux/irq.h index 38709a3..7f1f015 100644 --- a/include/linux/irq.h +++ b/include/linux/irq.h @@ -704,10 +704,12 @@ struct irq_chip_generic { * @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 * the parent irq. Usually GPIO implementations + * @IRQ_GC_MASK_CACHE_PER_TYPE: Mask cache is chip type private */ enum irq_gc_flags { IRQ_GC_INIT_MASK_CACHE = 1 << 0, IRQ_GC_INIT_NESTED_LOCK = 1 << 1, + IRQ_GC_MASK_CACHE_PER_TYPE = 1 << 2, }; /* Generic chip callback functions */ diff --git a/kernel/irq/generic-chip.c b/kernel/irq/generic-chip.c index 113d9eb..da2a941 100644 --- a/kernel/irq/generic-chip.c +++ b/kernel/irq/generic-chip.c @@ -241,18 +241,21 @@ void irq_setup_generic_chip(struct irq_chip_generic *gc, u32 msk, { struct irq_chip_type *ct = gc->chip_types; unsigned int i; + u32 *mskptr = &gc->mask_cache, mskreg = ct->regs.mask; raw_spin_lock(&gc_lock); list_add_tail(&gc->list, &gc_list); raw_spin_unlock(&gc_lock); - /* Init mask cache ? */ - if (flags & IRQ_GC_INIT_MASK_CACHE) - gc->mask_cache = irq_reg_readl(gc->reg_base + ct->regs.mask); - - /* Initialize mask cache pointer */ - for (i = 0; i < gc->num_ct; i++) - ct[i].mask_cache = &gc->mask_cache; + for (i = 0; i < gc->num_ct; i++) { + if (flags & IRQ_GC_MASK_CACHE_PER_TYPE) { + mskptr = &ct[i].mask_cache_priv; + mskreg = ct[i].regs.mask; + } + ct[i].mask_cache = mskptr; + if (flags & IRQ_GC_INIT_MASK_CACHE) + *mskptr = irq_reg_readl(gc->reg_base + mskreg); + } for (i = gc->irq_base; msk; msk >>= 1, i++) { if (!(msk & 0x01))