Re: [PATCH] dt-bindings: usb: add missing compatible arraylist

[Karan Sanghavi]

On Mon, 23 Sept 2024 at 17:23, Roger Quadros <rogerq@xxxxxxxxxx> wrote:
>
> On 22/09/2024 20:00, Karan Sanghavi wrote:
> > Added the vice versa order for compatible property in the yaml file so
> > that the dtb can parse for the order mentioned in the dts file
> > k3-am642-sk.dts for ti,j721e-usb.yaml
>
> k3-am642-sk.dts does not introduce any nodes with the said compatibles.
>
> "ti,am64-usb" compatible is introduced by k3-am642-main.dtsi.
> There is only one compatible introduced so there is nothing to do about
> order here.
>
> i.e.
>         usbss0: cdns-usb@f900000 {
>                 compatible = "ti,am64-usb";
>                 reg = <0x00 0xf900000 0x00 0x100>;
>
> What is the functional problem you are facing? Maybe then someone
> can point you in the right direction.
>

Sorry for mentioning the wrong file name; yes,
the problem is in k3-am64-main.dtsi file as mentioned below

 usbss0: cdns-usb@f900000 {
 766                 compatible = "ti,am64-usb", "ti,j721e-usb";
 767                 reg = <0x00 0xf900000 0x00 0x100>;

Due to this, it gives an error as in the YAML file the order is in
reverse order.
Also, I cloned the repo yesterday and still found the same error in
the dtsi file.
So have I done something wrong? as I see I might be out of sync with
latest changes

Thank you.
> >
> > This is highly ambiguous to me as where exactly the changes needs to be
> > added is it in the dts file or is the yaml where we have to reverse the
> > order already mentioned or do we have to add the another order as I have
> > done ?
> >
> > Signed-off-by: Karan Sanghavi <karansanghvi98@xxxxxxxxx>
> > ---
> >  Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml | 3 +++
> >  1 file changed, 3 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml b/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml
> > index 95ff9791baea..822653217c43 100644
> > --- a/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml
> > +++ b/Documentation/devicetree/bindings/usb/ti,j721e-usb.yaml
> > @@ -17,6 +17,9 @@ properties:
> >        - items:
> >            - const: ti,j721e-usb
> >            - const: ti,am64-usb
> > +      - items:
> > +          - const: ti,am64-usb
> > +          - const: ti,j721e-usb
> >
> >    reg:
> >      maxItems: 1
>
> --
> cheers,
> -roger