From: Yong Zhang <yong.zhang0@gmail.com>
To: linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org
Cc: tglx@linutronix.de, yong.zhang0@gmail.com,
David Howells <dhowells@redhat.com>,
Koichi Yasutake <yasutake.koichi@jp.panasonic.com>,
Mark Salter <msalter@redhat.com>,
John Stultz <johnstul@us.ibm.com>,
Kiyoshi Owada <owada.kiyoshi@jp.panasonic.com>,
Akira Takeuchi <takeuchi.akr@jp.panasonic.com>,
Andrew Morton <akpm@linux-foundation.org>,
Russell King <rmk+kernel@arm.linux.org.uk>,
Martin Schwidefsky <schwidefsky@de.ibm.com>,
linux-am33-list@redhat.com
Subject: [PATCH 13/55] mn10300: irq: Remove IRQF_DISABLED
Date: Thu, 22 Sep 2011 16:58:40 +0800 [thread overview]
Message-ID: <1316681962-8217-14-git-send-email-yong.zhang0@gmail.com> (raw)
In-Reply-To: <1316681962-8217-1-git-send-email-yong.zhang0@gmail.com>
Since commit [e58aa3d2: genirq: Run irq handlers with interrupts disabled],
We run all interrupt handlers with interrupts disabled
and we even check and yell when an interrupt handler
returns with interrupts enabled (see commit [b738a50a:
genirq: Warn when handler enables interrupts]).
So now this flag is a NOOP and can be removed.
Signed-off-by: Yong Zhang <yong.zhang0@gmail.com>
Acked-by: David Howells <dhowells@redhat.com>
---
arch/mn10300/kernel/cevt-mn10300.c | 2 +-
arch/mn10300/kernel/mn10300-serial.c | 6 +++---
arch/mn10300/kernel/smp.c | 1 -
arch/mn10300/unit-asb2364/irq-fpga.c | 2 +-
4 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/arch/mn10300/kernel/cevt-mn10300.c b/arch/mn10300/kernel/cevt-mn10300.c
index 69cae02..2415b76 100644
--- a/arch/mn10300/kernel/cevt-mn10300.c
+++ b/arch/mn10300/kernel/cevt-mn10300.c
@@ -103,7 +103,7 @@ int __init init_clockevents(void)
cd->set_next_event = next_event;
iact = &per_cpu(timer_irq, cpu);
- iact->flags = IRQF_DISABLED | IRQF_SHARED | IRQF_TIMER;
+ iact->flags = IRQF_SHARED | IRQF_TIMER;
iact->handler = timer_interrupt;
clockevents_register_device(cd);
diff --git a/arch/mn10300/kernel/mn10300-serial.c b/arch/mn10300/kernel/mn10300-serial.c
index 94901c5..6644c0d 100644
--- a/arch/mn10300/kernel/mn10300-serial.c
+++ b/arch/mn10300/kernel/mn10300-serial.c
@@ -936,15 +936,15 @@ static int mn10300_serial_startup(struct uart_port *_port)
irq_set_chip(port->tm_irq, &mn10300_serial_pic);
if (request_irq(port->rx_irq, mn10300_serial_interrupt,
- IRQF_DISABLED, port->rx_name, port) < 0)
+ 0, port->rx_name, port) < 0)
goto error;
if (request_irq(port->tx_irq, mn10300_serial_interrupt,
- IRQF_DISABLED, port->tx_name, port) < 0)
+ 0, port->tx_name, port) < 0)
goto error2;
if (request_irq(port->tm_irq, mn10300_serial_interrupt,
- IRQF_DISABLED, port->tm_name, port) < 0)
+ 0, port->tm_name, port) < 0)
goto error3;
mn10300_serial_mask_ack(port->tm_irq);
diff --git a/arch/mn10300/kernel/smp.c b/arch/mn10300/kernel/smp.c
index 9242e9f..6159ad8 100644
--- a/arch/mn10300/kernel/smp.c
+++ b/arch/mn10300/kernel/smp.c
@@ -142,7 +142,6 @@ static struct irqaction call_function_ipi = {
static irqreturn_t smp_ipi_timer_interrupt(int irq, void *dev_id);
static struct irqaction local_timer_ipi = {
.handler = smp_ipi_timer_interrupt,
- .flags = IRQF_DISABLED,
.name = "smp local timer IPI"
};
#endif
diff --git a/arch/mn10300/unit-asb2364/irq-fpga.c b/arch/mn10300/unit-asb2364/irq-fpga.c
index e16c216..073e2cc 100644
--- a/arch/mn10300/unit-asb2364/irq-fpga.c
+++ b/arch/mn10300/unit-asb2364/irq-fpga.c
@@ -76,7 +76,7 @@ static irqreturn_t fpga_interrupt(int irq, void *_mask)
static struct irqaction fpga_irq[] = {
[0] = {
.handler = fpga_interrupt,
- .flags = IRQF_DISABLED | IRQF_SHARED,
+ .flags = IRQF_SHARED,
.name = "fpga",
},
};
--
1.7.4.1
next prev parent reply other threads:[~2011-09-22 9:01 UTC|newest]
Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-22 8:58 [PATCH V2 00/55] Tree wide: cleanup IRQF_DISABLED Yong Zhang
2011-09-22 8:58 ` [PATCH 01/55] usb/hcd: Remove tricky code dealing with IRQF_DISABLED && IRQF_SHARED Yong Zhang
2011-09-22 8:58 ` [PATCH 02/55] alpha: irq: Remove IRQF_DISABLED Yong Zhang
2011-09-22 8:58 ` [PATCH 04/55] avr32: " Yong Zhang
2011-09-22 8:58 ` [PATCH 05/55] cris: " Yong Zhang
2011-09-22 8:58 ` [PATCH 06/55] frv: " Yong Zhang
2011-09-22 8:58 ` [PATCH 07/55] h8300: " Yong Zhang
2011-09-22 8:58 ` [PATCH 08/55] ia64: " Yong Zhang
2011-09-22 8:58 ` [PATCH 09/55] m32r: " Yong Zhang
2011-09-22 8:58 ` [PATCH 10/55] m68k: " Yong Zhang
2011-09-22 8:58 ` [PATCH 11/55] microblaze: " Yong Zhang
2011-09-22 8:58 ` Yong Zhang [this message]
2011-09-22 8:58 ` [PATCH 14/55] parisc: irq: remove IRQF_DISABLED Yong Zhang
2011-09-22 8:58 ` [PATCH 15/55] powerpc: irq: Remove IRQF_DISABLED Yong Zhang
2011-09-22 8:58 ` [PATCH 16/55] score: " Yong Zhang
2011-09-22 8:58 ` [PATCH 17/55] SH: " Yong Zhang
2011-09-22 8:58 ` [PATCH 18/55] sparc: " Yong Zhang
2011-09-22 8:58 ` [PATCH 19/55] um: " Yong Zhang
2011-09-22 10:46 ` Richard Weinberger
2011-09-22 8:58 ` [PATCH 20/55] unicore32: " Yong Zhang
2011-09-22 8:58 ` [PATCH 21/55] x86: irq: " Yong Zhang
2011-09-22 8:58 ` [PATCH 22/55] xtensa: " Yong Zhang
2011-09-22 8:58 ` [PATCH 23/55] clocksource: " Yong Zhang
2011-09-22 8:58 ` [PATCH 24/55] SCSI: " Yong Zhang
2011-09-23 1:35 ` Jack Wang
2011-09-22 8:58 ` [PATCH 25/55] ata: " Yong Zhang
2011-09-22 8:58 ` [PATCH 26/55] block: " Yong Zhang
2011-09-22 8:58 ` [PATCH 27/55] cdrom: " Yong Zhang
2011-09-22 8:58 ` [PATCH 28/55] driver/char: " Yong Zhang
2011-09-22 8:58 ` [PATCH 29/55] crypto: " Yong Zhang
2011-09-22 8:58 ` [PATCH 30/55] dmaengine: " Yong Zhang
2011-09-22 8:58 ` [PATCH 31/55] EDAC: " Yong Zhang
2011-09-22 8:58 ` [PATCH 32/55] i2c: " Yong Zhang
2011-09-24 11:53 ` Jean Delvare
2011-10-04 20:59 ` Ben Dooks
2011-09-22 8:59 ` [PATCH 33/55] infiniband: " Yong Zhang
2011-09-22 16:56 ` Roland Dreier
2011-09-23 1:40 ` Yong Zhang
2011-09-23 1:48 ` Yong Zhang
2011-09-23 18:09 ` Roland Dreier
2011-09-22 8:59 ` [PATCH 34/55] ISDN: " Yong Zhang
2011-09-22 8:59 ` [PATCH 36/55] drivers:misc:irq: " Yong Zhang
2011-09-22 8:59 ` [PATCH 37/55] mmc: irq: " Yong Zhang
2011-09-23 18:55 ` Chris Ball
2011-09-22 8:59 ` [PATCH 38/55] mtd: " Yong Zhang
2011-09-22 8:59 ` [PATCH 40/55] dirvers/parisc: " Yong Zhang
2011-09-22 8:59 ` [PATCH 41/55] pcmcia: " Yong Zhang
2011-09-22 8:59 ` [PATCH 42/55] sony-laptop:irq: " Yong Zhang
2011-09-22 8:59 ` [PATCH 43/55] PNP: irq: " Yong Zhang
2011-09-22 8:59 ` [PATCH 44/55] power_supply:irq: " Yong Zhang
2011-11-25 20:35 ` Anton Vorontsov
2011-09-22 8:59 ` [PATCH 45/55] powerpc/ps3: irq: " Yong Zhang
2011-09-22 8:59 ` [PATCH 46/55] rtc: " Yong Zhang
2011-09-22 8:59 ` [PATCH 47/55] spi: " Yong Zhang
2011-09-22 8:59 ` [PATCH 48/55] TTY: " Yong Zhang
2011-09-22 8:59 ` [PATCH 49/55] video: " Yong Zhang
2011-10-04 20:20 ` Florian Tobias Schandinat
2011-09-22 8:59 ` [PATCH 50/55] w1: " Yong Zhang
2011-09-22 8:59 ` [PATCH 51/55] watchdog: " Yong Zhang
2011-09-22 8:59 ` [PATCH 52/55] xen: " Yong Zhang
2011-09-22 8:59 ` [PATCH 54/55] Documentation: irq: Change documents related to IRQF_DISABLED Yong Zhang
2011-09-22 8:59 ` [PATCH 55/55] genirq: Remove IRQF_DISABLED Yong Zhang
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=1316681962-8217-14-git-send-email-yong.zhang0@gmail.com \
--to=yong.zhang0@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=dhowells@redhat.com \
--cc=johnstul@us.ibm.com \
--cc=linux-am33-list@redhat.com \
--cc=linux-arch@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=msalter@redhat.com \
--cc=owada.kiyoshi@jp.panasonic.com \
--cc=rmk+kernel@arm.linux.org.uk \
--cc=schwidefsky@de.ibm.com \
--cc=takeuchi.akr@jp.panasonic.com \
--cc=tglx@linutronix.de \
--cc=yasutake.koichi@jp.panasonic.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®