From: Shinya Kuribayashi Date: Sun, 27 Jun 2010 13:52:01 +0000 (+0900) Subject: MIPS: Loongson: irq.c: Misc cleanups X-Git-Tag: v2.6.36-rc1~563^2~72 X-Git-Url: https://openfabrics.org/gitweb/?a=commitdiff_plain;h=f5c1ca77a3b2f9b424c4f39bd71143d0ac308924;p=~shefty%2Frdma-dev.git MIPS: Loongson: irq.c: Misc cleanups * Remove unnecessary 'if (int_status & (1 <<10))' statement * s/if (foo != 0)/if (foo)/ * Remove unused 'inst_status &= ~(1 << i);' line Signed-off-by: Shinya Kuribayashi To: wuzhangjin@gmail.com CC: linux-mips@linux-mips.org Patchwork: https://patchwork.linux-mips.org/patch/1433/ Acked-by: Wu Zhangjin Signed-off-by: Ralf Baechle --- diff --git a/arch/mips/loongson/common/irq.c b/arch/mips/loongson/common/irq.c index 20e73283197..25a11df0348 100644 --- a/arch/mips/loongson/common/irq.c +++ b/arch/mips/loongson/common/irq.c @@ -21,19 +21,16 @@ void bonito_irqdispatch(void) /* workaround the IO dma problem: let cpu looping to allow DMA finish */ int_status = LOONGSON_INTISR; - if (int_status & (1 << 10)) { - while (int_status & (1 << 10)) { - udelay(1); - int_status = LOONGSON_INTISR; - } + while (int_status & (1 << 10)) { + udelay(1); + int_status = LOONGSON_INTISR; } /* Get pending sources, masked by current enables */ int_status = LOONGSON_INTISR & LOONGSON_INTEN; - if (int_status != 0) { + if (int_status) { i = __ffs(int_status); - int_status &= ~(1 << i); do_IRQ(LOONGSON_IRQ_BASE + i); } }