dt-bindings: usb: add missing compatible arraylist

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

Posted by Karan Sanghavi 2 months, 1 week ago

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

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@gmail.com>
---
 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
-- 
2.43.0

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

Posted by Roger Quadros 2 months ago

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.

> 
> 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@gmail.com>
> ---
>  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

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

Posted by Karan Sanghavi 2 months ago

On Mon, 23 Sept 2024 at 17:23, Roger Quadros <rogerq@kernel.org> 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@gmail.com>
> > ---
> >  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

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

Posted by Roger Quadros 2 months ago


On 28/09/2024 10:39, Karan Sanghavi wrote:
> On Mon, 23 Sept 2024 at 17:23, Roger Quadros <rogerq@kernel.org> 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>;
> 

Which tree are you looking at?

https://elixir.bootlin.com/linux/v6.11/source/arch/arm64/boot/dts/ti/k3-am64-main.dtsi#L761

	usbss0: cdns-usb@f900000 {
		compatible = "ti,am64-usb";
		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

You seem to be looking at some old tree. Please refer to latest kernel
tree when submitting changes upstream.

-- 
cheers,
-roger

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

Posted by Krzysztof Kozlowski 2 months, 1 week ago

On 22/09/2024 19: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
> 
> 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 ?

If you do not understand the problem being fixed thus not know how to
fix it, I don't quite get why you decided to send a patch...

> 
> Signed-off-by: Karan Sanghavi <karansanghvi98@gmail.com>
> ---
>  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

Sorry, this does not look right - duplicates above compatibles. Nothing
in the commit msg explains why this order is needed (in terms of hardware).

Best regards,
Krzysztof