From: Shubham Patil <shubhamsanjay.patil@amd.com>
To: <alexandre.belloni@bootlin.com>, <Frank.Li@kernel.org>,
<robh@kernel.org>, <krzk+dt@kernel.org>, <conor+dt@kernel.org>
Cc: <linux-i3c@lists.infradead.org>, <devicetree@vger.kernel.org>,
<linux-kernel@vger.kernel.org>, <git@amd.com>,
Shubham Patil <shubhamsanjay.patil@amd.com>
Subject: [PATCH v2 0/3] i3c: master: amd: Add IBI and hot-join support
Date: Mon, 24 Aug 2026 10:18:32 +0530 [thread overview]
Message-ID: <20260824044835.2924141-1-shubhamsanjay.patil@amd.com> (raw)
This series adds In-Band Interrupt (IBI) and Hot-Join support to the AMD
AXI I3C master controller driver.
An In-Band Interrupt (IBI) replaces the dedicated interrupt pin an I2C
slave would otherwise need. When the bus is idle, a target wanting
attention pulls SDA low to generate a START, then drives its own dynamic
address with RnW=1 during the arbitrated address header. The controller
either ACKs the request, accepting the interrupt and optionally reading a
Mandatory Data Byte and payload, or NACKs it. The interrupt therefore
travels in band on SDA/SCL, saving a wire per device.
Hot-Join (HJ) lets a target that was not present at boot join a running
bus, after being powered up later or physically attached. Having no
dynamic address yet, it arbitrates using the reserved Hot-Join address;
the controller ACKs that and runs dynamic address assignment to enumerate
it. The same hardware arbitrates and ACKs it as an IBI.
This series adds optional In-Band Interrupt (IBI) and Hot-Join support
to the Xilinx AXI I3C master driver. The controller is soft IP
synthesized into FPGA fabric, not fixed SoC hardware. IBI and Hot-Join
are Vivado synthesis-time options, so two bitstreams — or two instances
in the same design — can share the same IP version and compatible
string yet differ in which features are enabled. A single device may
contain multiple instances in mixed configurations. The compatible
string identifies the IP and its version, but not the capabilities of a
particular instance.
Patch 1 adds two boolean device tree properties,
xlnx,in-band-interrupt-capable and xlnx,hot-join-capable, to describe
what each instance implements. Hot-Join requests are ACKed by the IBI
machinery, so xlnx,hot-join-capable depends on
xlnx,in-band-interrupt-capable and the binding enforces that
dependency. The controller interrupt carries only IBI and Hot-Join
events, so it is required only for in-band-interrupt-capable designs.
The driver assembles its ops at probe time from these capability flags.
Patch 2 adds IBI support. The IBI ACK enable and its interrupt mask are
controller-wide rather than per-target, so they are armed on the first
target to enable IBIs and disarmed on the last to disable them, with
per-target control left on the bus via ENEC/DISEC. IBIs also share the
response and read FIFOs with normal transfers and are distinguished only
by the reserved TID 0x0F, so the controller stops ACKing IBIs for the
duration of a transfer and a target retries instead. One ACKed just
before that takes effect is found by TID while the transfer waits for its
own response, and is delivered through the normal IBI path.
Patch 3 adds Hot-Join. ENTDAA needs bus traffic and can sleep, so it
cannot run in the hard IRQ handler; the Hot-Join event is handed to the
i3c core with i3c_master_queue_hotjoin(), which runs the re-enumeration
from the core workqueue.
Changes in v2:
- Patch 1: Renamed the properties to "xlnx,in-band-interrupt-capable"
and "xlnx,hot-join-capable" and expanded their descriptions; expressed
the interrupt requirement with dependencies: instead of an
allOf/if-then clause.
- Patch 2: Do not lose an IBI raced with a transfer - the IBI ACK is now
held off for the duration of a transfer so the target retries, and one
accepted just before that is delivered through the normal IBI path
instead of being drained, which could also consume the transfer's read
data. disable_ibi() returns early on DISEC failure instead of
disarming and desyncing ibi.enabled_count from the core. Added
synchronize_irq() before disable_ibi() returns and before free_ibi()
frees the pool. Reworded commit message accordingly.
- Patch 3: Use the core's i3c_master_queue_hotjoin() instead of a
private work item, and only queue when the design is hot-join capable;
the core owns the work and cancels it on unregister. Reworded commit
message accordingly.
v1 : https://lore.kernel.org/all/20260814115155.3974988-1-shubhamsanjay.patil@amd.com/
Shubham Patil (3):
dt-bindings: i3c: xlnx: Add IBI and hot-join capability properties
i3c: master: amd: Add support for in-band interrupts
i3c: master: amd: Add hot-join support
.../bindings/i3c/xlnx,axi-i3c-1.0.yaml | 24 +
drivers/i3c/master/amd-i3c-master.c | 654 +++++++++++++++++-
2 files changed, 658 insertions(+), 20 deletions(-)
--
2.34.1
next reply other threads:[~2026-08-24 4:48 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-24 4:48 Shubham Patil [this message]
2026-08-24 4:48 ` [PATCH v2 1/3] dt-bindings: i3c: xlnx: Add IBI and hot-join capability properties Shubham Patil
2026-08-24 15:06 ` Frank Li
2026-08-24 16:36 ` Conor Dooley
2026-08-24 4:48 ` [PATCH v2 2/3] i3c: master: amd: Add support for in-band interrupts Shubham Patil
2026-08-24 5:04 ` sashiko-bot
2026-08-24 4:48 ` [PATCH v2 3/3] i3c: master: amd: Add hot-join support Shubham Patil
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=20260824044835.2924141-1-shubhamsanjay.patil@amd.com \
--to=shubhamsanjay.patil@amd.com \
--cc=Frank.Li@kernel.org \
--cc=alexandre.belloni@bootlin.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=git@amd.com \
--cc=krzk+dt@kernel.org \
--cc=linux-i3c@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robh@kernel.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