[PATCH v3] i3c: dw: Initialize spinlock to avoid upsetting lockdep

Fredrik Markstrom via B4 Relay posted 1 patch 3 weeks, 2 days ago
drivers/i3c/master/dw-i3c-master.c | 2 ++
1 file changed, 2 insertions(+)
[PATCH v3] i3c: dw: Initialize spinlock to avoid upsetting lockdep
Posted by Fredrik Markstrom via B4 Relay 3 weeks, 2 days ago
From: Fredrik Markstrom <fredrik.markstrom@est.tech>

The devs_lock spinlock introduced when adding support for ibi:s was
never initialized.

Fixes: e389b1d72a624 ("i3c: dw: Add support for in-band interrupts")
Suggested-by: Jani Nurminen <jani.nurminen@windriver.com>
Signed-off-by: Fredrik Markstrom <fredrik.markstrom@est.tech>
Reviewed-by: Ivar Holmqvist <ivar.holmqvist@est.tech>
---
Changes in v3:
- EDITME: describe what is new in this series revision.
- EDITME: use bulletpoints and terse descriptions.
- Link to v2: https://lore.kernel.org/r/20260116-i3c_dw_initialize_spinlock-v2-1-e56b431fb187@est.tech

Changes in v2:
- Added context in commit message
- Link to v1: https://lore.kernel.org/r/20260115-i3c_dw_initialize_spinlock-v1-1-b9c5bfa8be24@est.tech
---
 drivers/i3c/master/dw-i3c-master.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/i3c/master/dw-i3c-master.c b/drivers/i3c/master/dw-i3c-master.c
index 889e2ed5bc83..d5b070404056 100644
--- a/drivers/i3c/master/dw-i3c-master.c
+++ b/drivers/i3c/master/dw-i3c-master.c
@@ -1570,6 +1570,8 @@ int dw_i3c_common_probe(struct dw_i3c_master *master,
 	spin_lock_init(&master->xferqueue.lock);
 	INIT_LIST_HEAD(&master->xferqueue.list);
 
+	spin_lock_init(&master->devs_lock);
+
 	writel(INTR_ALL, master->regs + INTR_STATUS);
 	irq = platform_get_irq(pdev, 0);
 	ret = devm_request_irq(&pdev->dev, irq,

---
base-commit: 0f61b1860cc3f52aef9036d7235ed1f017632193
change-id: 20260115-i3c_dw_initialize_spinlock-d878212a846b

Best regards,
-- 
Fredrik Markstrom <fredrik.markstrom@est.tech>
Re: [PATCH v3] i3c: dw: Initialize spinlock to avoid upsetting lockdep
Posted by Alexandre Belloni 1 week, 1 day ago
On Fri, 16 Jan 2026 15:29:42 +0100, Fredrik Markstrom wrote:
> The devs_lock spinlock introduced when adding support for ibi:s was
> never initialized.
> 
> 

Applied, thanks!

[1/1] i3c: dw: Initialize spinlock to avoid upsetting lockdep
      https://git.kernel.org/i3c/c/b58eaa4761ab

Best regards,

-- 
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com