]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
spi: mpfs: fix controller deregistration
authorJohan Hovold <johan@kernel.org>
Thu, 9 Apr 2026 12:04:19 +0000 (14:04 +0200)
committerMark Brown <broonie@kernel.org>
Thu, 9 Apr 2026 19:08:14 +0000 (20:08 +0100)
Make sure to deregister the controller before disabling underlying
resources like interrupts during driver unbind.

Fixes: 9ac8d17694b6 ("spi: add support for microchip fpga spi controllers")
Cc: stable@vger.kernel.org # 6.0
Cc: Conor Dooley <conor.dooley@microchip.com>
Signed-off-by: Johan Hovold <johan@kernel.org>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
Link: https://patch.msgid.link/20260409120419.388546-21-johan@kernel.org
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-mpfs.c

index 64d15a6188ac7f3d801e2b1d2e0a7870c9ed4fbd..989a379b07008ed583b67e268fd2e362c45602ad 100644 (file)
@@ -574,7 +574,7 @@ static int mpfs_spi_probe(struct platform_device *pdev)
 
        mpfs_spi_init(host, spi);
 
-       ret = devm_spi_register_controller(&pdev->dev, host);
+       ret = spi_register_controller(host);
        if (ret) {
                mpfs_spi_disable_ints(spi);
                mpfs_spi_disable(spi);
@@ -592,6 +592,8 @@ static void mpfs_spi_remove(struct platform_device *pdev)
        struct spi_controller *host  = platform_get_drvdata(pdev);
        struct mpfs_spi *spi = spi_controller_get_devdata(host);
 
+       spi_unregister_controller(host);
+
        mpfs_spi_disable_ints(spi);
        mpfs_spi_disable(spi);
 }