From: Johan Hovold <johan@kernel.org>
To: Mark Brown <broonie@kernel.org>
Cc: Matthias Brugger <matthias.bgg@gmail.com>,
AngeloGioacchino Del Regno
<angelogioacchino.delregno@collabora.com>,
Frank Li <Frank.Li@nxp.com>,
Sascha Hauer <s.hauer@pengutronix.de>,
Andrew Jeffery <andrew@codeconstruct.com.au>,
Avi Fishman <avifishman70@gmail.com>,
Tomer Maimon <tmaimon77@gmail.com>,
Tali Perry <tali.perry1@gmail.com>,
Linus Walleij <linusw@kernel.org>,
Andi Shyti <andi.shyti@kernel.org>,
Tudor Ambarus <tudor.ambarus@linaro.org>,
Paul Walmsley <pjw@kernel.org>,
Samuel Holland <samuel.holland@sifive.com>,
Orson Zhai <orsonzhai@gmail.com>,
Baolin Wang <baolin.wang@linux.alibaba.com>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
Masahisa Kojima <kojima.masahisa@socionext.com>,
Jassi Brar <jassisinghbrar@gmail.com>,
Laxman Dewangan <ldewangan@nvidia.com>,
Kunihiko Hayashi <hayashi.kunihiko@socionext.com>,
Masami Hiramatsu <mhiramat@kernel.org>,
Michal Simek <michal.simek@amd.com>,
linux-spi@vger.kernel.org, linux-kernel@vger.kernel.org,
Johan Hovold <johan@kernel.org>,
stable@vger.kernel.org, Jingoo Han <jg1.han@samsung.com>
Subject: [PATCH 10/26] spi: rspi: fix controller deregistration
Date: Fri, 10 Apr 2026 10:17:40 +0200 [thread overview]
Message-ID: <20260410081757.503099-11-johan@kernel.org> (raw)
In-Reply-To: <20260410081757.503099-1-johan@kernel.org>
Make sure to deregister the controller before releasing underlying
resources like DMA during driver unbind.
Fixes: 9e03d05eee4c ("spi: rcar: Use devm_spi_register_master()")
Cc: stable@vger.kernel.org # 3.14
Cc: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: Johan Hovold <johan@kernel.org>
---
drivers/spi/spi-rspi.c | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/drivers/spi/spi-rspi.c b/drivers/spi/spi-rspi.c
index c739c1998b4c..a8180dece716 100644
--- a/drivers/spi/spi-rspi.c
+++ b/drivers/spi/spi-rspi.c
@@ -1171,8 +1171,14 @@ static void rspi_remove(struct platform_device *pdev)
{
struct rspi_data *rspi = platform_get_drvdata(pdev);
+ spi_controller_get(rspi->ctlr);
+
+ spi_unregister_controller(rspi->ctlr);
+
rspi_release_dma(rspi->ctlr);
pm_runtime_disable(&pdev->dev);
+
+ spi_controller_put(rspi->ctlr);
}
static const struct spi_ops rspi_ops = {
@@ -1376,9 +1382,9 @@ static int rspi_probe(struct platform_device *pdev)
if (ret < 0)
dev_warn(&pdev->dev, "DMA not available, using PIO\n");
- ret = devm_spi_register_controller(&pdev->dev, ctlr);
+ ret = spi_register_controller(ctlr);
if (ret < 0) {
- dev_err(&pdev->dev, "devm_spi_register_controller error.\n");
+ dev_err(&pdev->dev, "failed to register controller\n");
goto error3;
}
--
2.52.0
next prev parent reply other threads:[~2026-04-10 8:18 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20260410081757.503099-1-johan@kernel.org>
2026-04-10 8:17 ` [PATCH 01/26] spi: mt65xx: fix controller deregistration Johan Hovold
2026-04-10 8:17 ` [PATCH 02/26] spi: mtk-nor: " Johan Hovold
2026-04-10 8:17 ` [PATCH 03/26] spi: mxs: " Johan Hovold
2026-04-10 8:17 ` [PATCH 04/26] spi: npcm-pspi: " Johan Hovold
2026-04-10 8:17 ` [PATCH 05/26] spi: omap2-mcspi: " Johan Hovold
2026-04-10 8:17 ` [PATCH 06/26] spi: pic32: " Johan Hovold
2026-04-10 8:17 ` [PATCH 07/26] spi: pic32-sqi: " Johan Hovold
2026-04-10 8:17 ` [PATCH 08/26] spi: pl022: " Johan Hovold
2026-04-10 8:17 ` [PATCH 09/26] spi: qup: " Johan Hovold
2026-04-10 8:17 ` Johan Hovold [this message]
2026-04-10 8:17 ` [PATCH 11/26] spi: s3c64xx: " Johan Hovold
2026-04-10 8:17 ` [PATCH 12/26] spi: sh-hspi: " Johan Hovold
2026-04-10 8:17 ` [PATCH 13/26] spi: sh-msiof: " Johan Hovold
2026-04-10 8:17 ` [PATCH 14/26] spi: sifive: " Johan Hovold
2026-04-10 8:17 ` [PATCH 15/26] spi: slave-mt27xx: " Johan Hovold
2026-04-10 8:17 ` [PATCH 16/26] spi: sprd: " Johan Hovold
2026-04-10 8:17 ` [PATCH 17/26] spi: st-ssc4: " Johan Hovold
2026-04-10 8:17 ` [PATCH 18/26] spi: sun4i: " Johan Hovold
2026-04-10 8:17 ` [PATCH 19/26] spi: sun6i: " Johan Hovold
2026-04-10 8:17 ` [PATCH 20/26] spi: syncuacer: " Johan Hovold
2026-04-10 8:17 ` [PATCH 21/26] spi: tegra114: " Johan Hovold
2026-04-10 8:17 ` [PATCH 22/26] spi: tegra20-sflash: " Johan Hovold
2026-04-10 8:17 ` [PATCH 23/26] spi: ti-qspi: " Johan Hovold
2026-04-10 8:17 ` [PATCH 24/26] spi: uniphier: " Johan Hovold
2026-04-10 8:17 ` [PATCH 25/26] spi: zynqmp-gqspi: " Johan Hovold
2026-04-10 8:17 ` [PATCH 26/26] spi: zynq-qspi: " Johan Hovold
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260410081757.503099-11-johan@kernel.org \
--to=johan@kernel.org \
--cc=Frank.Li@nxp.com \
--cc=andi.shyti@kernel.org \
--cc=andrew@codeconstruct.com.au \
--cc=angelogioacchino.delregno@collabora.com \
--cc=avifishman70@gmail.com \
--cc=baolin.wang@linux.alibaba.com \
--cc=broonie@kernel.org \
--cc=hayashi.kunihiko@socionext.com \
--cc=jassisinghbrar@gmail.com \
--cc=jernej.skrabec@gmail.com \
--cc=jg1.han@samsung.com \
--cc=kojima.masahisa@socionext.com \
--cc=ldewangan@nvidia.com \
--cc=linusw@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-spi@vger.kernel.org \
--cc=matthias.bgg@gmail.com \
--cc=mhiramat@kernel.org \
--cc=michal.simek@amd.com \
--cc=orsonzhai@gmail.com \
--cc=pjw@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=samuel.holland@sifive.com \
--cc=stable@vger.kernel.org \
--cc=tali.perry1@gmail.com \
--cc=tmaimon77@gmail.com \
--cc=tudor.ambarus@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox