]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
ata: libahci_platform: fix IRQ check
authorSergey Shtylyov <s.shtylyov@omprussia.ru>
Mon, 15 Mar 2021 20:15:06 +0000 (23:15 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 22 May 2021 08:38:24 +0000 (10:38 +0200)
[ Upstream commit b30d0040f06159de97ad9c0b1536f47250719d7d ]

Iff platform_get_irq() returns 0, ahci_platform_init_host() would return 0
early (as if the call was successful). Override IRQ0 with -EINVAL instead
as the 'libata' regards 0 as "no IRQ" (thus polling) anyway...

Fixes: c034640a32f8 ("ata: libahci: properly propagate return value of platform_get_irq()")
Signed-off-by: Sergey Shtylyov <s.shtylyov@omprussia.ru>
Link: https://lore.kernel.org/r/4448c8cc-331f-2915-0e17-38ea34e251c8@omprussia.ru
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/ata/libahci_platform.c

index 65371e1befe8af32bf276a9a64fae183c1ca76b8..8839ad6b73e3be320bfe7d5a9d0fa37b5b6120c6 100644 (file)
@@ -516,11 +516,13 @@ int ahci_platform_init_host(struct platform_device *pdev,
        int i, irq, n_ports, rc;
 
        irq = platform_get_irq(pdev, 0);
-       if (irq <= 0) {
+       if (irq < 0) {
                if (irq != -EPROBE_DEFER)
                        dev_err(dev, "no irq\n");
                return irq;
        }
+       if (!irq)
+               return -EINVAL;
 
        hpriv->irq = irq;