spi: uniphier: Fix a bug that doesn't point to private data correctly
authorKunihiko Hayashi <hayashi.kunihiko@socionext.com>
Wed, 22 Dec 2021 04:48:12 +0000 (13:48 +0900)
committerMark Brown <broonie@kernel.org>
Wed, 26 Jan 2022 12:58:20 +0000 (12:58 +0000)
In uniphier_spi_remove(), there is a wrong code to get private data from
the platform device, so the driver can't be removed properly.

The driver should get spi_master from the platform device and retrieve
the private data from it.

Cc: <stable@vger.kernel.org>
Fixes: 5ba155a4d4cc ("spi: add SPI controller driver for UniPhier SoC")
Signed-off-by: Kunihiko Hayashi <hayashi.kunihiko@socionext.com>
Link: https://lore.kernel.org/r/1640148492-32178-1-git-send-email-hayashi.kunihiko@socionext.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-uniphier.c

index 8900e51..342ee8d 100644 (file)
@@ -767,12 +767,13 @@ out_master_put:
 
 static int uniphier_spi_remove(struct platform_device *pdev)
 {
-       struct uniphier_spi_priv *priv = platform_get_drvdata(pdev);
+       struct spi_master *master = platform_get_drvdata(pdev);
+       struct uniphier_spi_priv *priv = spi_master_get_devdata(master);
 
-       if (priv->master->dma_tx)
-               dma_release_channel(priv->master->dma_tx);
-       if (priv->master->dma_rx)
-               dma_release_channel(priv->master->dma_rx);
+       if (master->dma_tx)
+               dma_release_channel(master->dma_tx);
+       if (master->dma_rx)
+               dma_release_channel(master->dma_rx);
 
        clk_disable_unprepare(priv->clk);