Re: [PATCH v6 1/3] dt-bindings: i3c: Add support for Qualcomm I3C controller

[Mukesh Savaliya]

Hi Konrad, Apologies for the delayed response. I was waiting to verify 
the changes and ensure all the suggested updates were properly 
incorporated. The effort was further delayed due to some slave-side 
setup issues.

I have now addressed and validated all the suggested changes except one, 
which I have responded to separately below.

On 7/18/2025 2:21 AM, Konrad Dybcio wrote:
> On 7/1/25 9:18 AM, Mukesh Kumar Savaliya wrote:
> > Add device tree bindings for the Qualcomm I3C controller. This includes
> > the necessary documentation and properties required to describe the
> > hardware in the device tree.
> >
> > Signed-off-by: Mukesh Kumar Savaliya <mukesh.savaliya@xxxxxxxxxxxxxxxx>
> > Reviewed-by: Rob Herring (Arm) <robh@xxxxxxxxxx>
> > ---
>
> [...]
>
> > +properties:
> > +  compatible:
> > +    const: qcom,geni-i3c
> > +
> > +  reg:
> > +    maxItems: 1
>
> In another patchset, you expanded 'reg', can you coalesce that change
> into here, so that we don't have to worry about backwards compat?
Do you mean as below ?
reg:
  minItems: 1
  maxItems: 2

I did it earlier to add support of IBI feature, but i plan to do it 
later. Right now, i only want to add plain i3c controller support 
without IBI hence.

I shall add expanded reg while adding i3c support. For now, its not 
required. Please let me know if this is fine for now.

> Konrad