From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753031Ab1IVJEN (ORCPT ); Thu, 22 Sep 2011 05:04:13 -0400 Received: from mail-wy0-f174.google.com ([74.125.82.174]:33503 "EHLO mail-wy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753009Ab1IVJEJ (ORCPT ); Thu, 22 Sep 2011 05:04:09 -0400 From: Yong Zhang To: linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org Cc: tglx@linutronix.de, yong.zhang0@gmail.com, Vinod Koul , Dan Williams , Linus Walleij , linux-arm-kernel@lists.infradead.org Subject: [PATCH 30/55] dmaengine: irq: Remove IRQF_DISABLED Date: Thu, 22 Sep 2011 16:58:57 +0800 Message-Id: <1316681962-8217-31-git-send-email-yong.zhang0@gmail.com> X-Mailer: git-send-email 1.7.4.1 In-Reply-To: <1316681962-8217-1-git-send-email-yong.zhang0@gmail.com> References: <1316681962-8217-1-git-send-email-yong.zhang0@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 Acked-by: Vinod Koul Acked-by: Linus Walleij --- drivers/dma/amba-pl08x.c | 2 +- drivers/dma/coh901318.c | 2 +- drivers/dma/shdma.c | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/dma/amba-pl08x.c b/drivers/dma/amba-pl08x.c index b7cbd1a..896a119 100644 --- a/drivers/dma/amba-pl08x.c +++ b/drivers/dma/amba-pl08x.c @@ -1919,7 +1919,7 @@ static int pl08x_probe(struct amba_device *adev, const struct amba_id *id) writel(0x000000FF, pl08x->base + PL080_ERR_CLEAR); writel(0x000000FF, pl08x->base + PL080_TC_CLEAR); - ret = request_irq(adev->irq[0], pl08x_irq, IRQF_DISABLED, + ret = request_irq(adev->irq[0], pl08x_irq, 0, DRIVER_NAME, pl08x); if (ret) { dev_err(&adev->dev, "%s failed to request interrupt %d\n", diff --git a/drivers/dma/coh901318.c b/drivers/dma/coh901318.c index 4234f41..1dd9455 100644 --- a/drivers/dma/coh901318.c +++ b/drivers/dma/coh901318.c @@ -1506,7 +1506,7 @@ static int __init coh901318_probe(struct platform_device *pdev) if (irq < 0) goto err_no_irq; - err = request_irq(irq, dma_irq_handler, IRQF_DISABLED, + err = request_irq(irq, dma_irq_handler, 0, "coh901318", base); if (err) { dev_crit(&pdev->dev, diff --git a/drivers/dma/shdma.c b/drivers/dma/shdma.c index 7f49235..8c137d2 100644 --- a/drivers/dma/shdma.c +++ b/drivers/dma/shdma.c @@ -1106,7 +1106,7 @@ static void sh_dmae_chan_remove(struct sh_dmae_device *shdev) static int __init sh_dmae_probe(struct platform_device *pdev) { struct sh_dmae_pdata *pdata = pdev->dev.platform_data; - unsigned long irqflags = IRQF_DISABLED, + unsigned long irqflags = 0, chan_flag[SH_DMAC_MAX_CHANNELS] = {}; int errirq, chan_irq[SH_DMAC_MAX_CHANNELS]; int err, i, irq_cnt = 0, irqres = 0, irq_cap = 0; @@ -1266,7 +1266,7 @@ static int __init sh_dmae_probe(struct platform_device *pdev) IORESOURCE_IRQ_SHAREABLE) chan_flag[irq_cnt] = IRQF_SHARED; else - chan_flag[irq_cnt] = IRQF_DISABLED; + chan_flag[irq_cnt] = 0; dev_dbg(&pdev->dev, "Found IRQ %d for channel %d\n", i, irq_cnt); -- 1.7.4.1 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yong Zhang Subject: [PATCH 30/55] dmaengine: irq: Remove IRQF_DISABLED Date: Thu, 22 Sep 2011 16:58:57 +0800 Message-ID: <1316681962-8217-31-git-send-email-yong.zhang0@gmail.com> References: <1316681962-8217-1-git-send-email-yong.zhang0@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1316681962-8217-1-git-send-email-yong.zhang0@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-arm-kernel-bounces@lists.infradead.org Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org Cc: Linus Walleij , Vinod Koul , yong.zhang0@gmail.com, tglx@linutronix.de, Dan Williams , linux-arm-kernel@lists.infradead.org List-Id: linux-arch.vger.kernel.org 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 Acked-by: Vinod Koul Acked-by: Linus Walleij --- drivers/dma/amba-pl08x.c | 2 +- drivers/dma/coh901318.c | 2 +- drivers/dma/shdma.c | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/dma/amba-pl08x.c b/drivers/dma/amba-pl08x.c index b7cbd1a..896a119 100644 --- a/drivers/dma/amba-pl08x.c +++ b/drivers/dma/amba-pl08x.c @@ -1919,7 +1919,7 @@ static int pl08x_probe(struct amba_device *adev, const struct amba_id *id) writel(0x000000FF, pl08x->base + PL080_ERR_CLEAR); writel(0x000000FF, pl08x->base + PL080_TC_CLEAR); - ret = request_irq(adev->irq[0], pl08x_irq, IRQF_DISABLED, + ret = request_irq(adev->irq[0], pl08x_irq, 0, DRIVER_NAME, pl08x); if (ret) { dev_err(&adev->dev, "%s failed to request interrupt %d\n", diff --git a/drivers/dma/coh901318.c b/drivers/dma/coh901318.c index 4234f41..1dd9455 100644 --- a/drivers/dma/coh901318.c +++ b/drivers/dma/coh901318.c @@ -1506,7 +1506,7 @@ static int __init coh901318_probe(struct platform_device *pdev) if (irq < 0) goto err_no_irq; - err = request_irq(irq, dma_irq_handler, IRQF_DISABLED, + err = request_irq(irq, dma_irq_handler, 0, "coh901318", base); if (err) { dev_crit(&pdev->dev, diff --git a/drivers/dma/shdma.c b/drivers/dma/shdma.c index 7f49235..8c137d2 100644 --- a/drivers/dma/shdma.c +++ b/drivers/dma/shdma.c @@ -1106,7 +1106,7 @@ static void sh_dmae_chan_remove(struct sh_dmae_device *shdev) static int __init sh_dmae_probe(struct platform_device *pdev) { struct sh_dmae_pdata *pdata = pdev->dev.platform_data; - unsigned long irqflags = IRQF_DISABLED, + unsigned long irqflags = 0, chan_flag[SH_DMAC_MAX_CHANNELS] = {}; int errirq, chan_irq[SH_DMAC_MAX_CHANNELS]; int err, i, irq_cnt = 0, irqres = 0, irq_cap = 0; @@ -1266,7 +1266,7 @@ static int __init sh_dmae_probe(struct platform_device *pdev) IORESOURCE_IRQ_SHAREABLE) chan_flag[irq_cnt] = IRQF_SHARED; else - chan_flag[irq_cnt] = IRQF_DISABLED; + chan_flag[irq_cnt] = 0; dev_dbg(&pdev->dev, "Found IRQ %d for channel %d\n", i, irq_cnt); -- 1.7.4.1 From mboxrd@z Thu Jan 1 00:00:00 1970 From: yong.zhang0@gmail.com (Yong Zhang) Date: Thu, 22 Sep 2011 16:58:57 +0800 Subject: [PATCH 30/55] dmaengine: irq: Remove IRQF_DISABLED In-Reply-To: <1316681962-8217-1-git-send-email-yong.zhang0@gmail.com> References: <1316681962-8217-1-git-send-email-yong.zhang0@gmail.com> Message-ID: <1316681962-8217-31-git-send-email-yong.zhang0@gmail.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 Acked-by: Vinod Koul Acked-by: Linus Walleij --- drivers/dma/amba-pl08x.c | 2 +- drivers/dma/coh901318.c | 2 +- drivers/dma/shdma.c | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/dma/amba-pl08x.c b/drivers/dma/amba-pl08x.c index b7cbd1a..896a119 100644 --- a/drivers/dma/amba-pl08x.c +++ b/drivers/dma/amba-pl08x.c @@ -1919,7 +1919,7 @@ static int pl08x_probe(struct amba_device *adev, const struct amba_id *id) writel(0x000000FF, pl08x->base + PL080_ERR_CLEAR); writel(0x000000FF, pl08x->base + PL080_TC_CLEAR); - ret = request_irq(adev->irq[0], pl08x_irq, IRQF_DISABLED, + ret = request_irq(adev->irq[0], pl08x_irq, 0, DRIVER_NAME, pl08x); if (ret) { dev_err(&adev->dev, "%s failed to request interrupt %d\n", diff --git a/drivers/dma/coh901318.c b/drivers/dma/coh901318.c index 4234f41..1dd9455 100644 --- a/drivers/dma/coh901318.c +++ b/drivers/dma/coh901318.c @@ -1506,7 +1506,7 @@ static int __init coh901318_probe(struct platform_device *pdev) if (irq < 0) goto err_no_irq; - err = request_irq(irq, dma_irq_handler, IRQF_DISABLED, + err = request_irq(irq, dma_irq_handler, 0, "coh901318", base); if (err) { dev_crit(&pdev->dev, diff --git a/drivers/dma/shdma.c b/drivers/dma/shdma.c index 7f49235..8c137d2 100644 --- a/drivers/dma/shdma.c +++ b/drivers/dma/shdma.c @@ -1106,7 +1106,7 @@ static void sh_dmae_chan_remove(struct sh_dmae_device *shdev) static int __init sh_dmae_probe(struct platform_device *pdev) { struct sh_dmae_pdata *pdata = pdev->dev.platform_data; - unsigned long irqflags = IRQF_DISABLED, + unsigned long irqflags = 0, chan_flag[SH_DMAC_MAX_CHANNELS] = {}; int errirq, chan_irq[SH_DMAC_MAX_CHANNELS]; int err, i, irq_cnt = 0, irqres = 0, irq_cap = 0; @@ -1266,7 +1266,7 @@ static int __init sh_dmae_probe(struct platform_device *pdev) IORESOURCE_IRQ_SHAREABLE) chan_flag[irq_cnt] = IRQF_SHARED; else - chan_flag[irq_cnt] = IRQF_DISABLED; + chan_flag[irq_cnt] = 0; dev_dbg(&pdev->dev, "Found IRQ %d for channel %d\n", i, irq_cnt); -- 1.7.4.1