From 435d966dfd7e1776c1b974715cebbc5a96cfe36e Mon Sep 17 00:00:00 2001 From: Ondrej Jirman Date: Sun, 25 Feb 2024 04:22:51 +0100 Subject: spi fixes --- drivers/spi/spi-rockchip.c | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/drivers/spi/spi-rockchip.c b/drivers/spi/spi-rockchip.c index 878651f9c632..b874fe5762b6 100644 --- a/drivers/spi/spi-rockchip.c +++ b/drivers/spi/spi-rockchip.c @@ -781,7 +781,7 @@ static int rockchip_spi_enable_clocks(struct rockchip_spi *rs, bool en) clk_disable_unprepare(rs->apb_pclk); } - rs->clk_enabled = en; + rs->clk_enabled = !!en; return 0; } @@ -826,6 +826,7 @@ static int rockchip_spi_probe(struct platform_device *pdev) return dev_err_probe(&pdev->dev, PTR_ERR(rs->spiclk), "Failed to get spi_pclk\n"); + rs->clk_enabled = true; spi_enable_chip(rs, false); ret = platform_get_irq(pdev, 0); @@ -932,7 +933,9 @@ static int rockchip_spi_probe(struct platform_device *pdev) pm_runtime_set_autosuspend_delay(&pdev->dev, ROCKCHIP_AUTOSUSPEND_TIMEOUT); pm_runtime_use_autosuspend(&pdev->dev); pm_runtime_set_active(&pdev->dev); - pm_runtime_enable(&pdev->dev); + ret = devm_pm_runtime_enable(&pdev->dev); + if (ret < 0) + goto err_pm_disable; ret = devm_spi_register_controller(&pdev->dev, ctlr); if (ret < 0) { @@ -943,8 +946,6 @@ static int rockchip_spi_probe(struct platform_device *pdev) return 0; err_pm_disable: - pm_runtime_dont_use_autosuspend(&pdev->dev); - pm_runtime_disable(&pdev->dev); if (ctlr->dma_rx) dma_release_channel(ctlr->dma_rx); err_free_dma_tx: @@ -959,12 +960,6 @@ static void rockchip_spi_remove(struct platform_device *pdev) struct spi_controller *ctlr = spi_controller_get(platform_get_drvdata(pdev)); struct rockchip_spi *rs = spi_controller_get_devdata(ctlr); - pm_runtime_get_sync(&pdev->dev); - //pm_runtime_dont_use_autosuspend(&pdev->dev); - pm_runtime_put_noidle(&pdev->dev); - pm_runtime_disable(&pdev->dev); - pm_runtime_set_suspended(&pdev->dev); - if (ctlr->dma_tx) dma_release_channel(ctlr->dma_tx); if (ctlr->dma_rx) -- 2.35.3