diff options
| author | Álvaro Fernández Rojas <noltari@gmail.com> | 2020-05-18 09:19:06 +0200 |
|---|---|---|
| committer | Álvaro Fernández Rojas <noltari@gmail.com> | 2020-05-18 10:55:06 +0200 |
| commit | 8e2201ea5000467509c980ab1daf353e0ff98493 (patch) | |
| tree | b2ef5dab1919a8739c38a9c1e34dfe5c72a8baa4 | |
| parent | 183e9843e1b694cc6183de6855b897a85b0a2a36 (diff) | |
| download | openwrt-8e2201ea5000467509c980ab1daf353e0ff98493.tar.gz openwrt-8e2201ea5000467509c980ab1daf353e0ff98493.tar.bz2 openwrt-8e2201ea5000467509c980ab1daf353e0ff98493.zip | |
bcm63xx: ext_intc: fix warning
In file included from ./arch/mips/include/asm/io.h:34,
from ./arch/mips/include/asm/mmiowb.h:5,
from ./include/linux/spinlock.h:60,
from ./include/linux/irq.h:14,
from drivers/irqchip/irq-bcm6345-ext.c:10:
drivers/irqchip/irq-bcm6345-ext.c: In function 'bcm6345_ext_intc_of_init':
./arch/mips/include/asm/mach-bcm63xx/ioremap.h:48:9: warning: 'base' may be used uninitialized in this function [-Wmaybe-uninitialized]
return is_bcm63xx_internal_registers((unsigned long)addr);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/irqchip/irq-bcm6345-ext.c:255:16: note: 'base' was declared here
void __iomem *base;
^~~~
Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
(cherry picked from commit 29c3bb5f413f0962971c66e6516b8409a78e0135)
| -rw-r--r-- | target/linux/brcm63xx/patches-4.9/325-irqchip-bcm6345-external-fix-base-uninitialized.patch | 28 |
1 files changed, 28 insertions, 0 deletions
diff --git a/target/linux/brcm63xx/patches-4.9/325-irqchip-bcm6345-external-fix-base-uninitialized.patch b/target/linux/brcm63xx/patches-4.9/325-irqchip-bcm6345-external-fix-base-uninitialized.patch new file mode 100644 index 0000000000..45b5118a4a --- /dev/null +++ b/target/linux/brcm63xx/patches-4.9/325-irqchip-bcm6345-external-fix-base-uninitialized.patch @@ -0,0 +1,28 @@ +--- a/drivers/irqchip/irq-bcm6345-ext.c ++++ b/drivers/irqchip/irq-bcm6345-ext.c +@@ -271,21 +271,19 @@ static int __init bcm6345_ext_intc_of_in + + for (i = 0; i < num_irqs; i++) { + irqs[i] = irq_of_parse_and_map(node, i); +- if (!irqs[i]) { +- ret = -ENOMEM; +- goto out_unmap; +- } ++ if (!irqs[i]) ++ return -ENOMEM; + } + + base = of_iomap(node, 0); + if (!base) +- goto out_unmap; ++ return -ENXIO; + + ret = __bcm6345_ext_intc_init(node, num_irqs, irqs, base, shift, + toggle_clear_on_ack); + if (!ret) + return 0; +-out_unmap: ++ + iounmap(base); + + for (i = 0; i < num_irqs; i++) |