ARM: pxa: pxa_cplds: signedness bug in probe
authorRobert Jarzmik <robert.jarzmik@free.fr>
Thu, 21 May 2015 19:55:42 +0000 (21:55 +0200)
committerArnd Bergmann <arnd@arndb.de>
Fri, 22 May 2015 14:41:20 +0000 (16:41 +0200)
"base_irq" needs to be signed for the error handling to work.  Also we
can remove the initialization because we re-assign it later.

Fixes: aa8d6b73ea33 ('ARM: pxa: pxa_cplds: add lubbock and mainstone IO')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Robert Jarzmik <robert.jarzmik@free.fr>
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
arch/arm/mach-pxa/pxa_cplds_irqs.c

index f1aeb54..2385052 100644 (file)
@@ -107,7 +107,7 @@ static int cplds_probe(struct platform_device *pdev)
        struct resource *res;
        struct cplds *fpga;
        int ret;
-       unsigned int base_irq = 0;
+       int base_irq;
        unsigned long irqflags = 0;
 
        fpga = devm_kzalloc(&pdev->dev, sizeof(*fpga), GFP_KERNEL);