Skip to content

Commit c25c9f3

Browse files
jhovoldgregkh
authored andcommitted
spi: microchip-core-qspi: fix controller deregistration
commit e646414 upstream. Make sure to deregister the controller before disabling underlying resources like interrupts during driver unbind. Fixes: 8596124 ("spi: microchip-core-qspi: Add support for microchip fpga qspi controllers") Cc: stable@vger.kernel.org # 6.1 Cc: Naga Sureshkumar Relli <nagasuresh.relli@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-19-johan@kernel.org Signed-off-by: Mark Brown <broonie@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent 2ca3034 commit c25c9f3

1 file changed

Lines changed: 8 additions & 4 deletions

File tree

drivers/spi/spi-microchip-core-qspi.c

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -512,7 +512,7 @@ static int mchp_coreqspi_probe(struct platform_device *pdev)
512512
"unable to allocate host for QSPI controller\n");
513513

514514
qspi = spi_controller_get_devdata(ctlr);
515-
platform_set_drvdata(pdev, qspi);
515+
platform_set_drvdata(pdev, ctlr);
516516

517517
qspi->regs = devm_platform_ioremap_resource(pdev, 0);
518518
if (IS_ERR(qspi->regs))
@@ -545,7 +545,7 @@ static int mchp_coreqspi_probe(struct platform_device *pdev)
545545
SPI_TX_DUAL | SPI_TX_QUAD;
546546
ctlr->dev.of_node = np;
547547

548-
ret = devm_spi_register_controller(&pdev->dev, ctlr);
548+
ret = spi_register_controller(ctlr);
549549
if (ret)
550550
return dev_err_probe(&pdev->dev, ret,
551551
"spi_register_controller failed\n");
@@ -555,9 +555,13 @@ static int mchp_coreqspi_probe(struct platform_device *pdev)
555555

556556
static void mchp_coreqspi_remove(struct platform_device *pdev)
557557
{
558-
struct mchp_coreqspi *qspi = platform_get_drvdata(pdev);
559-
u32 control = readl_relaxed(qspi->regs + REG_CONTROL);
558+
struct spi_controller *ctlr = platform_get_drvdata(pdev);
559+
struct mchp_coreqspi *qspi = spi_controller_get_devdata(ctlr);
560+
u32 control;
560561

562+
spi_unregister_controller(ctlr);
563+
564+
control = readl_relaxed(qspi->regs + REG_CONTROL);
561565
mchp_coreqspi_disable_ints(qspi);
562566
control &= ~CONTROL_ENABLE;
563567
writel_relaxed(control, qspi->regs + REG_CONTROL);

0 commit comments

Comments
 (0)