]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
spi: pl022: switch to managed controller allocation
authorJohan Hovold <johan@kernel.org>
Tue, 5 May 2026 07:28:52 +0000 (09:28 +0200)
committerMark Brown <broonie@kernel.org>
Mon, 11 May 2026 00:55:40 +0000 (09:55 +0900)
Switch to device managed controller allocation to simplify error
handling and to avoid having to take another reference during
deregistration.

Signed-off-by: Johan Hovold <johan@kernel.org>
Reviewed-by: Linus Walleij <linusw@kernel.org>
Link: https://patch.msgid.link/20260505072909.618363-4-johan@kernel.org
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-pl022.c

index 9c0211f94fd07c47375d407065f7333986b05937..95652df5fd092db3f3613f9bcd4178e6a5d12cab 100644 (file)
@@ -1868,7 +1868,7 @@ static int pl022_probe(struct amba_device *adev, const struct amba_id *id)
        }
 
        /* Allocate host with space for data */
-       host = spi_alloc_host(dev, sizeof(struct pl022));
+       host = devm_spi_alloc_host(dev, sizeof(struct pl022));
        if (host == NULL) {
                dev_err(&adev->dev, "probe - cannot alloc SPI host\n");
                return -ENOMEM;
@@ -1907,7 +1907,7 @@ static int pl022_probe(struct amba_device *adev, const struct amba_id *id)
 
        status = amba_request_regions(adev, NULL);
        if (status)
-               goto err_no_ioregion;
+               return status;
 
        pl022->phybase = adev->res.start;
        pl022->virtbase = devm_ioremap(dev, adev->res.start,
@@ -1984,8 +1984,7 @@ static int pl022_probe(struct amba_device *adev, const struct amba_id *id)
  err_no_clk:
  err_no_ioremap:
        amba_release_regions(adev);
- err_no_ioregion:
-       spi_controller_put(host);
+
        return status;
 }
 
@@ -1997,8 +1996,6 @@ pl022_remove(struct amba_device *adev)
        if (!pl022)
                return;
 
-       spi_controller_get(pl022->host);
-
        spi_unregister_controller(pl022->host);
 
        /*
@@ -2012,8 +2009,6 @@ pl022_remove(struct amba_device *adev)
                pl022_dma_remove(pl022);
 
        amba_release_regions(adev);
-
-       spi_controller_put(pl022->host);
 }
 
 #ifdef CONFIG_PM_SLEEP