All of lore.kernel.org
 help / color / mirror / Atom feed
From: tip-bot for Grygorii Strashko <tipbot@zytor.com>
To: linux-tip-commits@vger.kernel.org
Cc: balbi@ti.com, grygorii.strashko@ti.com,
	linux-kernel@vger.kernel.org, tony@atomide.com,
	marc.zyngier@arm.com, linux-arm-kernel@lists.infradead.org,
	hpa@zytor.com, jason@lakedaemon.net, sudeep.holla@arm.com,
	tglx@linutronix.de, linux@arm.linux.org.uk, mingo@kernel.org,
	nsekhar@ti.com
Subject: [tip:irq/urgent] ARM: OMAP: wakeupgen: Restore the irq_set_type() mechanism
Date: Wed, 19 Aug 2015 16:22:38 -0700	[thread overview]
Message-ID: <tip-63059a272398ef5dc1bd7065a036e8b6e82d1af7@git.kernel.org> (raw)
In-Reply-To: <1439554830-19502-5-git-send-email-grygorii.strashko@ti.com>

Commit-ID:  63059a272398ef5dc1bd7065a036e8b6e82d1af7
Gitweb:     http://git.kernel.org/tip/63059a272398ef5dc1bd7065a036e8b6e82d1af7
Author:     Grygorii Strashko <grygorii.strashko@ti.com>
AuthorDate: Fri, 14 Aug 2015 15:20:28 +0300
Committer:  Thomas Gleixner <tglx@linutronix.de>
CommitDate: Thu, 20 Aug 2015 00:25:25 +0200

ARM: OMAP: wakeupgen: Restore the irq_set_type() mechanism

The conversion of the wakeupgen irqchip to hierarchical irq domains
failed to provide a mechanism to properly set the trigger type of an
interrupt.

The wakeupgen irq chip itself has no mechanism and therefor no
irq_set_type() callback. The code before the conversion relayed the
trigger configuration directly to the underlying GIC.

Restore the correct behaviour by setting the wakeupgen irq_set_type
callback to irq_chip_set_type_parent(). This propagates the
set_trigger() call to the underlying GIC irqchip.

[ tglx: Massaged changelog ]

Fixes: 7136d457f365 ('ARM: omap: convert wakeupgen to stacked domains')
Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
Acked-by: Tony Lindgren <tony@atomide.com>
Cc: Sudeep Holla <sudeep.holla@arm.com>
Cc: <linux@arm.linux.org.uk>
Cc: <nsekhar@ti.com>
Cc: <jason@lakedaemon.net>
Cc: <balbi@ti.com>
Cc: <linux-arm-kernel@lists.infradead.org>
Cc: <marc.zyngier@arm.com>
Cc: stable@vger.kernel.org # 4.1
Link: http://lkml.kernel.org/r/1439554830-19502-5-git-send-email-grygorii.strashko@ti.com
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
---
 arch/arm/mach-omap2/omap-wakeupgen.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/mach-omap2/omap-wakeupgen.c b/arch/arm/mach-omap2/omap-wakeupgen.c
index 8e52621..e1d2e99 100644
--- a/arch/arm/mach-omap2/omap-wakeupgen.c
+++ b/arch/arm/mach-omap2/omap-wakeupgen.c
@@ -392,6 +392,7 @@ static struct irq_chip wakeupgen_chip = {
 	.irq_mask		= wakeupgen_mask,
 	.irq_unmask		= wakeupgen_unmask,
 	.irq_retrigger		= irq_chip_retrigger_hierarchy,
+	.irq_set_type		= irq_chip_set_type_parent,
 	.flags			= IRQCHIP_SKIP_SET_WAKE | IRQCHIP_MASK_ON_SUSPEND,
 #ifdef CONFIG_SMP
 	.irq_set_affinity	= irq_chip_set_affinity_parent,

  reply	other threads:[~2015-08-19 23:23 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-14 12:20 [PATCH v3 0/6] genirq: irqdomain_hierarchy: fixes Grygorii Strashko
2015-08-14 12:20 ` Grygorii Strashko
2015-08-14 12:20 ` Grygorii Strashko
2015-08-14 12:20 ` [PATCH v3 1/6] genirq: fix irq_chip_retrigger_hierarchy Grygorii Strashko
2015-08-14 12:20   ` Grygorii Strashko
2015-08-14 12:20   ` Grygorii Strashko
2015-08-17  3:28   ` Jiang Liu
2015-08-17  3:28     ` Jiang Liu
2015-08-19 23:21   ` [tip:irq/urgent] genirq: Don' t return ENOSYS in irq_chip_retrigger_hierarchy tip-bot for Grygorii Strashko
2015-08-14 12:20 ` [PATCH v3 2/6] genirq: introduce irq_chip_set_type_parent() helper Grygorii Strashko
2015-08-14 12:20   ` Grygorii Strashko
2015-08-14 12:20   ` Grygorii Strashko
2015-08-19 23:21   ` [tip:irq/urgent] genirq: Introduce " tip-bot for Grygorii Strashko
2015-08-14 12:20 ` [PATCH v3 3/6] irqchip: crossbar: fix arm gic irq type configuration Grygorii Strashko
2015-08-14 12:20   ` Grygorii Strashko
2015-08-14 12:20   ` Grygorii Strashko
2015-08-19 23:22   ` [tip:irq/urgent] irqchip/crossbar: Restore the irq_set_type() mechanism tip-bot for Grygorii Strashko
2015-08-14 12:20 ` [PATCH v3 4/6] ARM: OMAP: wakeupgen: fix arm gic irq type configuration Grygorii Strashko
2015-08-14 12:20   ` Grygorii Strashko
2015-08-14 12:20   ` Grygorii Strashko
2015-08-19 23:22   ` tip-bot for Grygorii Strashko [this message]
2015-08-14 12:20 ` [PATCH v3 5/6] irqchip: crossbar: fix irq masking at suspend Grygorii Strashko
2015-08-14 12:20   ` Grygorii Strashko
2015-08-14 12:20   ` Grygorii Strashko
2015-08-19 23:22   ` [tip:irq/urgent] irqchip/crossbar: Restore the mask on suspend behaviour tip-bot for Grygorii Strashko
2015-08-14 12:20 ` [PATCH v3 6/6] irqchip: crossbar: fix set_wake functionality Grygorii Strashko
2015-08-14 12:20   ` Grygorii Strashko
2015-08-14 12:20   ` Grygorii Strashko
2015-08-19 23:23   ` [tip:irq/urgent] irqchip/crossbar: Restore " tip-bot for Grygorii Strashko

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=tip-63059a272398ef5dc1bd7065a036e8b6e82d1af7@git.kernel.org \
    --to=tipbot@zytor.com \
    --cc=balbi@ti.com \
    --cc=grygorii.strashko@ti.com \
    --cc=hpa@zytor.com \
    --cc=jason@lakedaemon.net \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-tip-commits@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=marc.zyngier@arm.com \
    --cc=mingo@kernel.org \
    --cc=nsekhar@ti.com \
    --cc=sudeep.holla@arm.com \
    --cc=tglx@linutronix.de \
    --cc=tony@atomide.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.