spi: uniphier: Simplify clock handling with devm_clk_get_enabled()

Replace devm_clk_get() followed by clk_prepare_enable() with
devm_clk_get_enabled() for the clock. This removes the need for
explicit clock enable and disable calls, as the managed API automatically
handles clock disabling on device removal or probe failure.

Remove the now-unnecessary clk_disable_unprepare() calls from the probe
error path and the remove callback. Adjust error labels accordingly.

Signed-off-by: Pei Xiao <xiaopei01@kylinos.cn>
Reviewed-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
Link: https://patch.msgid.link/b2deeefd4ef1a4bce71116aabfcb7e81400f6d37.1775546948.git.xiaopei01@kylinos.cn
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/drivers/spi/spi-uniphier.c b/drivers/spi/spi-uniphier.c
index 9e1d364a6198155cca39d5ddc9ed296f4ea56050..1b815ee2ed1b6e8c1e8a0d0156bcffa40ec7ad30 100644 (file)
--- a/drivers/spi/spi-uniphier.c
+++ b/drivers/spi/spi-uniphier.c
@@ -666,28 +666,24 @@ static int uniphier_spi_probe(struct platform_device *pdev)
        }
        priv->base_dma_addr = res->start;
 
-       priv->clk = devm_clk_get(&pdev->dev, NULL);
+       priv->clk = devm_clk_get_enabled(&pdev->dev, NULL);
        if (IS_ERR(priv->clk)) {
                dev_err(&pdev->dev, "failed to get clock\n");
                ret = PTR_ERR(priv->clk);
                goto out_host_put;
        }
 
-       ret = clk_prepare_enable(priv->clk);
-       if (ret)
-               goto out_host_put;
-
        irq = platform_get_irq(pdev, 0);
        if (irq < 0) {
                ret = irq;
-               goto out_disable_clk;
+               goto out_host_put;
        }
 
        ret = devm_request_irq(&pdev->dev, irq, uniphier_spi_handler,
                               0, "uniphier-spi", priv);
        if (ret) {
                dev_err(&pdev->dev, "failed to request IRQ\n");
-               goto out_disable_clk;
+               goto out_host_put;
        }
 
        init_completion(&priv->xfer_done);
@@ -716,7 +712,7 @@ static int uniphier_spi_probe(struct platform_device *pdev)
        if (IS_ERR_OR_NULL(host->dma_tx)) {
                if (PTR_ERR(host->dma_tx) == -EPROBE_DEFER) {
                        ret = -EPROBE_DEFER;
-                       goto out_disable_clk;
+                       goto out_host_put;
                }
                host->dma_tx = NULL;
                dma_tx_burst = INT_MAX;
@@ -766,9 +762,6 @@ out_release_dma:
                host->dma_tx = NULL;
        }
 
-out_disable_clk:
-       clk_disable_unprepare(priv->clk);
-
 out_host_put:
        spi_controller_put(host);
        return ret;
@@ -777,14 +770,11 @@ out_host_put:
 static void uniphier_spi_remove(struct platform_device *pdev)
 {
        struct spi_controller *host = platform_get_drvdata(pdev);
-       struct uniphier_spi_priv *priv = spi_controller_get_devdata(host);
 
        if (host->dma_tx)
                dma_release_channel(host->dma_tx);
        if (host->dma_rx)
                dma_release_channel(host->dma_rx);
-
-       clk_disable_unprepare(priv->clk);
 }
 
 static const struct of_device_id uniphier_spi_match[] = {