[PATCH] SPI: meson-spifc add missed calls to remove function
Mark Brown
broonie at kernel.org
Fri Apr 23 12:47:35 BST 2021
On Fri, Apr 23, 2021 at 11:42:47AM +0800, Artem Lapkin wrote:
> Problem: rmmod meson_gx_mmc - not stable without spi_master_suspend call
> and we can get stuck when try unload this module
> +++ b/drivers/spi/spi-meson-spifc.c
> @@ -359,6 +359,7 @@ static int meson_spifc_remove(struct platform_device *pdev)
> struct spi_master *master = platform_get_drvdata(pdev);
> struct meson_spifc *spifc = spi_master_get_devdata(master);
>
> + spi_master_suspend(master);
> pm_runtime_get_sync(&pdev->dev);
> clk_disable_unprepare(spifc->clk);
> pm_runtime_disable(&pdev->dev);
I would expect the driver to unregister the controller at the start of
the remove function, suspend doesn't really make sense here.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-amlogic/attachments/20210423/f4966c05/attachment.sig>
More information about the linux-amlogic
mailing list