> @@ -909,10 +901,6 @@ static int a3700_spi_probe(struct platform_device *pdev) > > static void a3700_spi_remove(struct platform_device *pdev) > { > - struct spi_controller *host = platform_get_drvdata(pdev); > - struct a3700_spi *spi = spi_controller_get_devdata(host); > - > - clk_unprepare(spi->clk); Hi. No point in keeping an empty remove function. Drop the whole thing now there is nothing to do. Jonathan > } > > static struct platform_driver a3700_spi_driver = {