Linux-i3c Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Frank Li <Frank.li@oss.nxp.com>
To: "Patil, Shubham Sanjay" <shubpati@amd.com>
Cc: Shubham Patil <shubhamsanjay.patil@amd.com>,
	Alexandre Belloni <alexandre.belloni@bootlin.com>,
	Frank Li <Frank.Li@nxp.com>, Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	linux-i3c@lists.infradead.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org, "git (AMD-Xilinx)" <git@amd.com>
Subject: Re: [PATCH 1/3] dt-bindings: i3c: xlnx: Add IBI and hot-join capability properties
Date: Tue, 18 Aug 2026 11:51:58 -0500	[thread overview]
Message-ID: <aoSNrlDOLNXg1Xfv@SMW015318> (raw)
In-Reply-To: <c608fce5-872b-43d7-a6cf-f9e49956dd42@amd.com>

On Tue, Aug 18, 2026 at 05:29:19PM +0530, Patil, Shubham Sanjay wrote:
> [You don't often get email from shubpati@amd.com. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]
>
> Hi Frank,
>
> On 8/14/2026 10:37 PM, Frank Li wrote:
> > [You don't often get email from frank.li@oss.nxp.com. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]
> >
> > Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding.
> >
> >
> > On Fri, Aug 14, 2026 at 05:21:53PM +0530, Shubham Patil wrote:
> > > In-Band Interrupt and Hot-Join are synthesis-time options of the AXI I3C
> > > IP. Describe them with two boolean properties.
> > >
> > > A Hot-Join request is acknowledged by the IBI machinery, so a hot-join
> > > capable design is always IBI capable as well. Both events are reported
> > > through the controller interrupt, which is therefore required whenever
> > > the capability is present.
> > >
> > > Signed-off-by: Shubham Patil <shubhamsanjay.patil@amd.com>
> > > ---
> > >   .../bindings/i3c/xlnx,axi-i3c-1.0.yaml        | 23 +++++++++++++++++++
> > >   1 file changed, 23 insertions(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/i3c/xlnx,axi-i3c-1.0.yaml b/Documentation/devicetree/bindings/i3c/xlnx,axi-i3c-1.0.yaml
> > > index 2caa245a8656..07e3d0b4d767 100644
> > > --- a/Documentation/devicetree/bindings/i3c/xlnx,axi-i3c-1.0.yaml
> > > +++ b/Documentation/devicetree/bindings/i3c/xlnx,axi-i3c-1.0.yaml
> > > @@ -32,13 +32,34 @@ properties:
> > >     interrupts:
> > >       maxItems: 1
> > >
> > > +  xlnx,ibi-capable:
> > > +    type: boolean
> > > +    description:
> > > +      The IP is synthesized with In-Band Interrupt support. IBIs are reported
> > > +      through the controller interrupt.
> > > +
> > > +  xlnx,hj-capable:
> > > +    type: boolean
> > > +    description:
> > > +      The IP is synthesized with Hot-Join support. A Hot-Join request is
> > > +      acknowledged by the same machinery as an In-Band Interrupt.
> > > +
> >
> > The compatiblity xlnx,axi-i3c-1.0 already indicates such cap. Do you have
> > variance. such as
> >
> >          one Soc have two instances.
> >          instance 1  have hj and ibi cap
> >          instance 2 no hj and ibi cap
>
> Yes, exactly that. The AXI I3C core is not a hard block: it is soft IP
> instantiated in the FPGA fabric, and IBI and Hot-Join are per-instance
> parameters selected in Vivado when the design is built.


I suggest put it i3c.yaml

capable-ibi:
  - enum: ["default", "enable", "disable"] to overwrite auto detect from
hardware for difference instances case. put default here to allow overlay
dtso to overwrite it since overlay can't delete property.

I suppose not only xlnx have such problems.

Anyways, need dt team to provide suggestions

Frank


>
> >
> > If new version support HJ and IBI, need use new compatible string.
>
> The compatible does track the IP version, and a future AXI I3C v2.0
> would get its own string. But IBI and Hot-Join are not version
> differences: they are independent build time options of this same v1.0
> core, so putting them in the compatible would mean several strings for
> one IP version, growing combinatorially with every future option.
>
> Thanks,
> Shubham>
> > Frank
> >
> > >   required:
> > >     - compatible
> > >     - reg
> > >     - clocks
> > >
> > > +dependencies:
> > > +  xlnx,hj-capable: ["xlnx,ibi-capable"]
> > > +
> > >   allOf:
> > >     - $ref: i3c.yaml#
> > > +  - if:
> > > +      required:
> > > +        - xlnx,ibi-capable
> > > +    then:
> > > +      required:
> > > +        - interrupts
> > >
> > >   unevaluatedProperties: false
> > >
> > > @@ -54,5 +75,7 @@ examples:
> > >           interrupts = <GIC_SPI 89 IRQ_TYPE_LEVEL_HIGH>;
> > >           #address-cells = <3>;
> > >           #size-cells = <0>;
> > > +        xlnx,ibi-capable;
> > > +        xlnx,hj-capable;
> > >       };
> > >   ...
> > > --
> > > 2.34.1
> > >
>

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

  reply	other threads:[~2026-08-18 16:52 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-14 11:51 [PATCH 0/3] i3c: master: amd: Add IBI and hot-join support Shubham Patil
2026-08-14 11:51 ` [PATCH 1/3] dt-bindings: i3c: xlnx: Add IBI and hot-join capability properties Shubham Patil
2026-08-14 15:38   ` Conor Dooley
2026-08-18 10:38     ` Patil, Shubham Sanjay
2026-08-14 17:07   ` Frank Li
2026-08-18 11:59     ` Patil, Shubham Sanjay
2026-08-18 16:51       ` Frank Li [this message]
2026-08-17  8:50   ` Krzysztof Kozlowski
2026-08-18 12:02     ` Patil, Shubham Sanjay
2026-08-18 18:36       ` Krzysztof Kozlowski
2026-08-14 11:51 ` [PATCH 2/3] i3c: master: amd: Add support for in-band interrupts Shubham Patil
2026-08-14 12:05   ` sashiko-bot
2026-08-14 11:51 ` [PATCH 3/3] i3c: master: amd: Add hot-join support Shubham Patil
2026-08-14 12:05   ` sashiko-bot

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=aoSNrlDOLNXg1Xfv@SMW015318 \
    --to=frank.li@oss.nxp.com \
    --cc=Frank.Li@nxp.com \
    --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 \
    --cc=shubhamsanjay.patil@amd.com \
    --cc=shubpati@amd.com \
    /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