Commit 89e87730 authored by Jingoo Han's avatar Jingoo Han Committed by Mark Brown

spi: remove unnecessary platform_set_drvdata()

The driver core clears the driver data to NULL after device_release
or on probe failure, since commit 0998d063
(device-core: Ensure drvdata = NULL when no driver is bound).
Thus, it is not needed to manually clear the device driver data to NULL.
Signed-off-by: default avatarJingoo Han <jg1.han@samsung.com>
Acked-by: default avatarGrant Likely <grant.likely@linaro.org>
Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
parent f722406f
...@@ -285,7 +285,6 @@ static int altera_spi_probe(struct platform_device *pdev) ...@@ -285,7 +285,6 @@ static int altera_spi_probe(struct platform_device *pdev)
exit_busy: exit_busy:
err = -EBUSY; err = -EBUSY;
exit: exit:
platform_set_drvdata(pdev, NULL);
spi_master_put(master); spi_master_put(master);
return err; return err;
} }
...@@ -296,7 +295,6 @@ static int altera_spi_remove(struct platform_device *dev) ...@@ -296,7 +295,6 @@ static int altera_spi_remove(struct platform_device *dev)
struct spi_master *master = hw->bitbang.master; struct spi_master *master = hw->bitbang.master;
spi_bitbang_stop(&hw->bitbang); spi_bitbang_stop(&hw->bitbang);
platform_set_drvdata(dev, NULL);
spi_master_put(master); spi_master_put(master);
return 0; return 0;
} }
......
...@@ -287,7 +287,6 @@ static int ath79_spi_probe(struct platform_device *pdev) ...@@ -287,7 +287,6 @@ static int ath79_spi_probe(struct platform_device *pdev)
err_unmap: err_unmap:
iounmap(sp->base); iounmap(sp->base);
err_put_master: err_put_master:
platform_set_drvdata(pdev, NULL);
spi_master_put(sp->bitbang.master); spi_master_put(sp->bitbang.master);
return ret; return ret;
...@@ -302,7 +301,6 @@ static int ath79_spi_remove(struct platform_device *pdev) ...@@ -302,7 +301,6 @@ static int ath79_spi_remove(struct platform_device *pdev)
clk_disable(sp->clk); clk_disable(sp->clk);
clk_put(sp->clk); clk_put(sp->clk);
iounmap(sp->base); iounmap(sp->base);
platform_set_drvdata(pdev, NULL);
spi_master_put(sp->bitbang.master); spi_master_put(sp->bitbang.master);
return 0; return 0;
......
...@@ -987,8 +987,6 @@ static int au1550_spi_remove(struct platform_device *pdev) ...@@ -987,8 +987,6 @@ static int au1550_spi_remove(struct platform_device *pdev)
au1xxx_dbdma_chan_free(hw->dma_tx_ch); au1xxx_dbdma_chan_free(hw->dma_tx_ch);
} }
platform_set_drvdata(pdev, NULL);
spi_master_put(hw->master); spi_master_put(hw->master);
return 0; return 0;
} }
......
...@@ -469,7 +469,6 @@ static int bcm63xx_spi_probe(struct platform_device *pdev) ...@@ -469,7 +469,6 @@ static int bcm63xx_spi_probe(struct platform_device *pdev)
out_clk_disable: out_clk_disable:
clk_disable_unprepare(clk); clk_disable_unprepare(clk);
out_err: out_err:
platform_set_drvdata(pdev, NULL);
spi_master_put(master); spi_master_put(master);
out_clk: out_clk:
clk_put(clk); clk_put(clk);
...@@ -491,8 +490,6 @@ static int bcm63xx_spi_remove(struct platform_device *pdev) ...@@ -491,8 +490,6 @@ static int bcm63xx_spi_remove(struct platform_device *pdev)
clk_disable_unprepare(bs->clk); clk_disable_unprepare(bs->clk);
clk_put(bs->clk); clk_put(bs->clk);
platform_set_drvdata(pdev, 0);
spi_master_put(master); spi_master_put(master);
return 0; return 0;
......
...@@ -882,9 +882,6 @@ static int bfin_sport_spi_remove(struct platform_device *pdev) ...@@ -882,9 +882,6 @@ static int bfin_sport_spi_remove(struct platform_device *pdev)
peripheral_free_list(drv_data->pin_req); peripheral_free_list(drv_data->pin_req);
/* Prevent double remove */
platform_set_drvdata(pdev, NULL);
return 0; return 0;
} }
......
...@@ -1418,9 +1418,6 @@ static int bfin_spi_remove(struct platform_device *pdev) ...@@ -1418,9 +1418,6 @@ static int bfin_spi_remove(struct platform_device *pdev)
peripheral_free_list(drv_data->pin_req); peripheral_free_list(drv_data->pin_req);
/* Prevent double remove */
platform_set_drvdata(pdev, NULL);
return 0; return 0;
} }
......
...@@ -254,7 +254,6 @@ static int spi_clps711x_probe(struct platform_device *pdev) ...@@ -254,7 +254,6 @@ static int spi_clps711x_probe(struct platform_device *pdev)
if (gpio_is_valid(hw->chipselect[i])) if (gpio_is_valid(hw->chipselect[i]))
gpio_free(hw->chipselect[i]); gpio_free(hw->chipselect[i]);
platform_set_drvdata(pdev, NULL);
spi_master_put(master); spi_master_put(master);
kfree(master); kfree(master);
...@@ -274,7 +273,6 @@ static int spi_clps711x_remove(struct platform_device *pdev) ...@@ -274,7 +273,6 @@ static int spi_clps711x_remove(struct platform_device *pdev)
gpio_free(hw->chipselect[i]); gpio_free(hw->chipselect[i]);
devm_clk_put(&pdev->dev, hw->spi_clk); devm_clk_put(&pdev->dev, hw->spi_clk);
platform_set_drvdata(pdev, NULL);
spi_unregister_master(master); spi_unregister_master(master);
kfree(master); kfree(master);
......
...@@ -524,7 +524,6 @@ static int mcfqspi_remove(struct platform_device *pdev) ...@@ -524,7 +524,6 @@ static int mcfqspi_remove(struct platform_device *pdev)
/* disable the hardware (set the baud rate to 0) */ /* disable the hardware (set the baud rate to 0) */
mcfqspi_wr_qmr(mcfqspi, MCFQSPI_QMR_MSTR); mcfqspi_wr_qmr(mcfqspi, MCFQSPI_QMR_MSTR);
platform_set_drvdata(pdev, NULL);
mcfqspi_cs_teardown(mcfqspi); mcfqspi_cs_teardown(mcfqspi);
clk_disable(mcfqspi->clk); clk_disable(mcfqspi->clk);
clk_put(mcfqspi->clk); clk_put(mcfqspi->clk);
......
...@@ -111,8 +111,6 @@ static int dw_spi_mmio_remove(struct platform_device *pdev) ...@@ -111,8 +111,6 @@ static int dw_spi_mmio_remove(struct platform_device *pdev)
struct dw_spi_mmio *dwsmmio = platform_get_drvdata(pdev); struct dw_spi_mmio *dwsmmio = platform_get_drvdata(pdev);
struct resource *mem; struct resource *mem;
platform_set_drvdata(pdev, NULL);
clk_disable(dwsmmio->clk); clk_disable(dwsmmio->clk);
clk_put(dwsmmio->clk); clk_put(dwsmmio->clk);
dwsmmio->clk = NULL; dwsmmio->clk = NULL;
......
...@@ -1132,7 +1132,6 @@ static int ep93xx_spi_probe(struct platform_device *pdev) ...@@ -1132,7 +1132,6 @@ static int ep93xx_spi_probe(struct platform_device *pdev)
clk_put(espi->clk); clk_put(espi->clk);
fail_release_master: fail_release_master:
spi_master_put(master); spi_master_put(master);
platform_set_drvdata(pdev, NULL);
return error; return error;
} }
...@@ -1167,7 +1166,6 @@ static int ep93xx_spi_remove(struct platform_device *pdev) ...@@ -1167,7 +1166,6 @@ static int ep93xx_spi_remove(struct platform_device *pdev)
ep93xx_spi_release_dma(espi); ep93xx_spi_release_dma(espi);
clk_put(espi->clk); clk_put(espi->clk);
platform_set_drvdata(pdev, NULL);
spi_unregister_master(master); spi_unregister_master(master);
return 0; return 0;
......
...@@ -514,8 +514,6 @@ static int spi_gpio_remove(struct platform_device *pdev) ...@@ -514,8 +514,6 @@ static int spi_gpio_remove(struct platform_device *pdev)
status = spi_bitbang_stop(&spi_gpio->bitbang); status = spi_bitbang_stop(&spi_gpio->bitbang);
spi_master_put(spi_gpio->bitbang.master); spi_master_put(spi_gpio->bitbang.master);
platform_set_drvdata(pdev, NULL);
if (SPI_MISO_GPIO != SPI_GPIO_NO_MISO) if (SPI_MISO_GPIO != SPI_GPIO_NO_MISO)
gpio_free(SPI_MISO_GPIO); gpio_free(SPI_MISO_GPIO);
if (SPI_MOSI_GPIO != SPI_GPIO_NO_MOSI) if (SPI_MOSI_GPIO != SPI_GPIO_NO_MOSI)
......
...@@ -902,7 +902,6 @@ static int spi_imx_probe(struct platform_device *pdev) ...@@ -902,7 +902,6 @@ static int spi_imx_probe(struct platform_device *pdev)
} }
spi_master_put(master); spi_master_put(master);
kfree(master); kfree(master);
platform_set_drvdata(pdev, NULL);
return ret; return ret;
} }
...@@ -929,8 +928,6 @@ static int spi_imx_remove(struct platform_device *pdev) ...@@ -929,8 +928,6 @@ static int spi_imx_remove(struct platform_device *pdev)
release_mem_region(res->start, resource_size(res)); release_mem_region(res->start, resource_size(res));
platform_set_drvdata(pdev, NULL);
return 0; return 0;
} }
......
...@@ -368,7 +368,6 @@ static int tiny_spi_probe(struct platform_device *pdev) ...@@ -368,7 +368,6 @@ static int tiny_spi_probe(struct platform_device *pdev)
exit_busy: exit_busy:
err = -EBUSY; err = -EBUSY;
exit: exit:
platform_set_drvdata(pdev, NULL);
spi_master_put(master); spi_master_put(master);
return err; return err;
} }
...@@ -382,7 +381,6 @@ static int tiny_spi_remove(struct platform_device *pdev) ...@@ -382,7 +381,6 @@ static int tiny_spi_remove(struct platform_device *pdev)
spi_bitbang_stop(&hw->bitbang); spi_bitbang_stop(&hw->bitbang);
for (i = 0; i < hw->gpio_cs_count; i++) for (i = 0; i < hw->gpio_cs_count; i++)
gpio_free(hw->gpio_cs[i]); gpio_free(hw->gpio_cs[i]);
platform_set_drvdata(pdev, NULL);
spi_master_put(master); spi_master_put(master);
return 0; return 0;
} }
......
...@@ -1299,9 +1299,6 @@ static int pxa2xx_spi_remove(struct platform_device *pdev) ...@@ -1299,9 +1299,6 @@ static int pxa2xx_spi_remove(struct platform_device *pdev)
/* Disconnect from the SPI framework */ /* Disconnect from the SPI framework */
spi_unregister_master(drv_data->master); spi_unregister_master(drv_data->master);
/* Prevent double remove */
platform_set_drvdata(pdev, NULL);
return 0; return 0;
} }
......
...@@ -1399,7 +1399,6 @@ static int s3c64xx_spi_probe(struct platform_device *pdev) ...@@ -1399,7 +1399,6 @@ static int s3c64xx_spi_probe(struct platform_device *pdev)
err2: err2:
clk_disable_unprepare(sdd->clk); clk_disable_unprepare(sdd->clk);
err0: err0:
platform_set_drvdata(pdev, NULL);
spi_master_put(master); spi_master_put(master);
return ret; return ret;
...@@ -1420,7 +1419,6 @@ static int s3c64xx_spi_remove(struct platform_device *pdev) ...@@ -1420,7 +1419,6 @@ static int s3c64xx_spi_remove(struct platform_device *pdev)
clk_disable_unprepare(sdd->clk); clk_disable_unprepare(sdd->clk);
platform_set_drvdata(pdev, NULL);
spi_master_put(master); spi_master_put(master);
return 0; return 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment