ata: pata_ixp4xx: Remove unnecessary return value check
authorYangtao Li <frank.li@vivo.com>
Mon, 10 Jul 2023 02:45:48 +0000 (10:45 +0800)
committerDamien Le Moal <dlemoal@kernel.org>
Wed, 2 Aug 2023 08:37:06 +0000 (17:37 +0900)
As commit ce753ad1549c ("platform: finally disallow IRQ0 in
platform_get_irq() and its ilk") says, there is no need to
check if the platform_get_irq return value is 0. Let's remove it.

Signed-off-by: Yangtao Li <frank.li@vivo.com>
Reviewed-by: Sergey Shtylyov <s.shtylyov@omp.ru>
Signed-off-by: Damien Le Moal <dlemoal@kernel.org>
drivers/ata/pata_ixp4xx_cf.c

index 1b9f67e16864d056a6b681bf99008584aa2b1c36..246bb4f8f1f7f6fa451f9ddd1e2adb381cc962d4 100644 (file)
@@ -274,12 +274,9 @@ static int ixp4xx_pata_probe(struct platform_device *pdev)
                return PTR_ERR(ixpp->ctl);
 
        irq = platform_get_irq(pdev, 0);
-       if (irq > 0)
-               irq_set_irq_type(irq, IRQ_TYPE_EDGE_RISING);
-       else if (irq < 0)
+       if (irq < 0)
                return irq;
-       else
-               return -EINVAL;
+       irq_set_irq_type(irq, IRQ_TYPE_EDGE_RISING);
 
        /* Just one port to set up */
        ixp4xx_setup_port(ixpp->host->ports[0], ixpp, cmd->start, ctl->start);