fbdev: mb862xx: Fix check of return value from irq_of_parse_and_map()
authorChristophe Leroy <christophe.leroy@csgroup.eu>
Thu, 6 Oct 2022 05:33:17 +0000 (07:33 +0200)
committerHelge Deller <deller@gmx.de>
Sat, 8 Oct 2022 13:20:09 +0000 (15:20 +0200)
NO_IRQ is used to check the return of irq_of_parse_and_map().
On some architecture NO_IRQ is 0, on other architectures it is -1.
irq_of_parse_and_map() returns 0 on error, independent of NO_IRQ.
So use 0 instead of using NO_IRQ.

Signed-off-by: Christophe Leroy <christophe.leroy@csgroup.eu>
Signed-off-by: Helge Deller <deller@gmx.de>
drivers/video/fbdev/mb862xx/mb862xxfbdrv.c

index 96800c9..90c79e8 100644 (file)
@@ -693,7 +693,7 @@ static int of_platform_mb862xx_probe(struct platform_device *ofdev)
        par->dev = dev;
 
        par->irq = irq_of_parse_and_map(np, 0);
-       if (par->irq == NO_IRQ) {
+       if (!par->irq) {
                dev_err(dev, "failed to map irq\n");
                ret = -ENODEV;
                goto fbrel;