public inbox for linux-i3c@lists.infradead.org
 help / color / mirror / Atom feed
From: Adrian Hunter <adrian.hunter@intel.com>
To: alexandre.belloni@bootlin.com
Cc: Frank.Li@nxp.com, linux-i3c@lists.infradead.org
Subject: [PATCH 02/12] i3c: mipi-i3c-hci: Fix Hot-Join NACK
Date: Fri, 27 Feb 2026 16:11:39 +0200	[thread overview]
Message-ID: <20260227141149.184980-3-adrian.hunter@intel.com> (raw)
In-Reply-To: <20260227141149.184980-1-adrian.hunter@intel.com>

The MIPI I3C HCI host controller driver does not implement Hot-Join
handling, yet Hot-Join response control defaults to allowing devices to
Hot‑Join the bus.  Configure HC_CONTROL_HOT_JOIN_CTRL to NACK all Hot‑Join
attempts.

Fixes: 9ad9a52cce282 ("i3c/master: introduce the mipi-i3c-hci driver")
Cc: stable@vger.kernel.org
Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
---
 drivers/i3c/master/mipi-i3c-hci/core.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/i3c/master/mipi-i3c-hci/core.c b/drivers/i3c/master/mipi-i3c-hci/core.c
index dbe93df0c70e..4877a321edf9 100644
--- a/drivers/i3c/master/mipi-i3c-hci/core.c
+++ b/drivers/i3c/master/mipi-i3c-hci/core.c
@@ -152,7 +152,8 @@ static int i3c_hci_bus_init(struct i3c_master_controller *m)
 	if (hci->quirks & HCI_QUIRK_RESP_BUF_THLD)
 		amd_set_resp_buf_thld(hci);
 
-	reg_set(HC_CONTROL, HC_CONTROL_BUS_ENABLE);
+	/* Enable bus with Hot-Join disabled */
+	reg_set(HC_CONTROL, HC_CONTROL_BUS_ENABLE | HC_CONTROL_HOT_JOIN_CTRL);
 	dev_dbg(&hci->master.dev, "HC_CONTROL = %#x", reg_read(HC_CONTROL));
 
 	return 0;
@@ -764,7 +765,8 @@ static int i3c_hci_runtime_resume(struct device *dev)
 
 	hci->io->resume(hci);
 
-	reg_set(HC_CONTROL, HC_CONTROL_BUS_ENABLE);
+	/* Enable bus with Hot-Join disabled */
+	reg_set(HC_CONTROL, HC_CONTROL_BUS_ENABLE | HC_CONTROL_HOT_JOIN_CTRL);
 
 	return 0;
 }
-- 
2.51.0


-- 
linux-i3c mailing list
linux-i3c@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-i3c

  parent reply	other threads:[~2026-02-27 14:12 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-27 14:11 [PATCH 00/12] i3c: mipi-i3c-hci: Fixes for v7.0 Adrian Hunter
2026-02-27 14:11 ` [PATCH 01/12] i3c: mipi-i3c-hci: Use ETIMEDOUT instead of ETIME for timeout errors Adrian Hunter
2026-02-27 16:00   ` Frank Li
2026-02-27 14:11 ` Adrian Hunter [this message]
2026-02-27 16:03   ` [PATCH 02/12] i3c: mipi-i3c-hci: Fix Hot-Join NACK Frank Li
2026-03-02  8:42     ` Adrian Hunter
2026-03-02 17:52       ` Frank Li
2026-02-27 14:11 ` [PATCH 03/12] i3c: mipi-i3c-hci: Fix race in DMA ring enqueue for parallel xfers Adrian Hunter
2026-02-27 16:09   ` Frank Li
2026-02-27 14:11 ` [PATCH 04/12] i3c: mipi-i3c-hci: Fix race in DMA ring dequeue Adrian Hunter
2026-02-27 16:18   ` Frank Li
2026-03-02  8:43     ` Adrian Hunter
2026-03-02 19:23       ` Frank Li
2026-03-04 17:58         ` Adrian Hunter
2026-02-27 14:11 ` [PATCH 05/12] i3c: mipi-i3c-hci: Fix race between DMA ring dequeue and the interrupt handler Adrian Hunter
2026-02-27 16:21   ` Frank Li
2026-02-27 14:11 ` [PATCH 06/12] i3c: mipi-i3c-hci: Correct RING_CTRL_ABORT handling in DMA dequeue Adrian Hunter
2026-02-27 16:25   ` Frank Li
2026-03-02  8:45     ` Adrian Hunter
2026-03-02 17:49       ` Frank Li
2026-02-27 14:11 ` [PATCH 07/12] i3c: mipi-i3c-hci: Add missing TID field to no-op command descriptor Adrian Hunter
2026-02-27 16:27   ` Frank Li
2026-02-27 14:11 ` [PATCH 08/12] i3c: mipi-i3c-hci: Restart DMA ring correctly after dequeue abort Adrian Hunter
2026-02-27 16:28   ` Frank Li
2026-02-27 14:11 ` [PATCH 09/12] i3c: mipi-i3c-hci: Consolidate common xfer processing logic Adrian Hunter
2026-02-27 16:30   ` Frank Li
2026-02-27 14:11 ` [PATCH 10/12] i3c: mipi-i3c-hci: Fix race in DMA error handling in interrupt context Adrian Hunter
2026-02-27 16:40   ` Frank Li
2026-03-02  8:45     ` Adrian Hunter
2026-02-27 14:11 ` [PATCH 11/12] i3c: mipi-i3c-hci: Fix handling of shared IRQs during early initialization Adrian Hunter
2026-02-27 16:42   ` Frank Li
2026-03-02  8:43     ` Adrian Hunter
2026-03-02 19:18       ` Frank Li
2026-03-04 18:13         ` Adrian Hunter
2026-02-27 14:11 ` [PATCH 12/12] i3c: mipi-i3c-hci: Fallback to software reset when bus disable fails Adrian Hunter
2026-02-27 16:44   ` Frank Li

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=20260227141149.184980-3-adrian.hunter@intel.com \
    --to=adrian.hunter@intel.com \
    --cc=Frank.Li@nxp.com \
    --cc=alexandre.belloni@bootlin.com \
    --cc=linux-i3c@lists.infradead.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