]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
spi: Deduplicate deferred probe checks in spi_probe()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Sun, 8 Dec 2024 16:03:17 +0000 (18:03 +0200)
committerMark Brown <broonie@kernel.org>
Mon, 9 Dec 2024 13:14:20 +0000 (13:14 +0000)
Deduplicate deferred probe checks in spi_probe() and enable
the error message for ACPI case as well.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://patch.msgid.link/20241208195635.1271656-3-andriy.shevchenko@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi.c

index 88f785b9e6ec3e9008f48763764dfc548a6225ee..e0f79773be700252f8186d928586146a196504e2 100644 (file)
@@ -417,19 +417,14 @@ static int spi_probe(struct device *dev)
        if (ret)
                return ret;
 
-       if (is_of_node(fwnode)) {
+       if (is_of_node(fwnode))
                spi->irq = of_irq_get(dev->of_node, 0);
-               if (spi->irq == -EPROBE_DEFER)
-                       return dev_err_probe(dev, -EPROBE_DEFER, "Failed to get irq\n");
-               if (spi->irq < 0)
-                       spi->irq = 0;
-       } else if (is_acpi_device_node(fwnode) && spi->irq < 0) {
+       else if (is_acpi_device_node(fwnode) && spi->irq < 0)
                spi->irq = acpi_dev_gpio_irq_get(to_acpi_device_node(fwnode), 0);
-               if (spi->irq == -EPROBE_DEFER)
-                       return -EPROBE_DEFER;
-               if (spi->irq < 0)
-                       spi->irq = 0;
-       }
+       if (spi->irq == -EPROBE_DEFER)
+               return dev_err_probe(dev, spi->irq, "Failed to get irq\n");
+       if (spi->irq < 0)
+               spi->irq = 0;
 
        ret = dev_pm_domain_attach(dev, true);
        if (ret)