ata: pata_mpc52xx: Replace NO_IRQ with 0
authorChristophe Leroy <christophe.leroy@csgroup.eu>
Thu, 6 Oct 2022 05:17:07 +0000 (07:17 +0200)
committerDamien Le Moal <damien.lemoal@opensource.wdc.com>
Mon, 17 Oct 2022 02:38:15 +0000 (11:38 +0900)
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>
Reviewed-by: Sergey Shtylyov <s.shtylyov@omp.ru>
Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
drivers/ata/pata_mpc52xx.c

index 6559b60..3ebd652 100644 (file)
@@ -731,7 +731,7 @@ static int mpc52xx_ata_probe(struct platform_device *op)
                udma_mask = ATA_UDMA2 & ((1 << (*prop + 1)) - 1);
 
        ata_irq = irq_of_parse_and_map(op->dev.of_node, 0);
-       if (ata_irq == NO_IRQ) {
+       if (!ata_irq) {
                dev_err(&op->dev, "error mapping irq\n");
                return -EINVAL;
        }