summaryrefslogtreecommitdiffstats
path: root/drivers/spi/spi-fsl-lpspi.c
diff options
context:
space:
mode:
authorAxel Lin <axel.lin@ingics.com>2019-04-07 16:58:15 +0200
committerMark Brown <broonie@kernel.org>2019-04-08 09:07:30 +0200
commit6599be346daf9250b9fddfdea30a1d4779538730 (patch)
tree1854033243dd305a6cc7745661633bb4973b3dc6 /drivers/spi/spi-fsl-lpspi.c
parentspi-mem: fix kernel-doc for spi_mem_dirmap_{read|write}() (diff)
downloadlinux-6599be346daf9250b9fddfdea30a1d4779538730.tar.xz
linux-6599be346daf9250b9fddfdea30a1d4779538730.zip
spi: fsl-lpspi: Fix problematic dev_set_drvdata call
The original code already set controller as drvdata: platform_set_drvdata(pdev, controller); But commit 944c01a889d9 ("spi: lpspi: enable runtime pm for lpspi") added dev_set_drvdata(&pdev->dev, fsl_lpspi); so fsl_lpspi_remove() will get wrong pointer by platform_get_drvdata(). Fixes: 944c01a889d9 ("spi: lpspi: enable runtime pm for lpspi") Signed-off-by: Axel Lin <axel.lin@ingics.com> Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/spi/spi-fsl-lpspi.c')
-rw-r--r--drivers/spi/spi-fsl-lpspi.c11
1 files changed, 8 insertions, 3 deletions
diff --git a/drivers/spi/spi-fsl-lpspi.c b/drivers/spi/spi-fsl-lpspi.c
index 33b6a8affd55..a5c6f27666f3 100644
--- a/drivers/spi/spi-fsl-lpspi.c
+++ b/drivers/spi/spi-fsl-lpspi.c
@@ -788,9 +788,12 @@ static irqreturn_t fsl_lpspi_isr(int irq, void *dev_id)
static int fsl_lpspi_runtime_resume(struct device *dev)
{
- struct fsl_lpspi_data *fsl_lpspi = dev_get_drvdata(dev);
+ struct spi_controller *controller = dev_get_drvdata(dev);
+ struct fsl_lpspi_data *fsl_lpspi;
int ret;
+ fsl_lpspi = spi_controller_get_devdata(controller);
+
ret = clk_prepare_enable(fsl_lpspi->clk_per);
if (ret)
return ret;
@@ -806,7 +809,10 @@ static int fsl_lpspi_runtime_resume(struct device *dev)
static int fsl_lpspi_runtime_suspend(struct device *dev)
{
- struct fsl_lpspi_data *fsl_lpspi = dev_get_drvdata(dev);
+ struct spi_controller *controller = dev_get_drvdata(dev);
+ struct fsl_lpspi_data *fsl_lpspi;
+
+ fsl_lpspi = spi_controller_get_devdata(controller);
clk_disable_unprepare(fsl_lpspi->clk_per);
clk_disable_unprepare(fsl_lpspi->clk_ipg);
@@ -853,7 +859,6 @@ static int fsl_lpspi_probe(struct platform_device *pdev)
fsl_lpspi = spi_controller_get_devdata(controller);
fsl_lpspi->dev = &pdev->dev;
- dev_set_drvdata(&pdev->dev, fsl_lpspi);
fsl_lpspi->is_slave = of_property_read_bool((&pdev->dev)->of_node,
"spi-slave");