Re: [PATCH v2 1/2] dt-bindings: phy: ti,phy-usb3: Add aspcie reference clock setting

[Rob Herring]

On Mon, Aug 17, 2026 at 07:19:51AM +0200, David Oberhollenzer wrote:
> Hi,
> 
> On 8/12/26 4:20 AM, Rob Herring wrote:
> > > +  - if:
> > > +      properties:
> > > +        compatible:
> > > +          contains:
> > > +            const: ti,phy-pipe3-pcie
> > > +    then:
> > > +      properties:
> > > +        ti,syscon-acspcie-tx-rx: true
> > 
> > This has no effect. Invert the if and move the else here.
> > 
> 
> Before I send another version of this patch set, are you really
> sure about this?

Yes. The only purpose it serves is you have to have a 'then' to have an 
'else' clause. (BTW, json-schema actually allows omitting 'then' and 
having just if/else, but the dtschema meta-schema does not.)

> 
> Not only did I copy that entire block from the one directly above,

It's wrong too.

> (the one that is supposed to only allow syscon-pllreset compatible
> is set to ti,phy-pipe3-sata), I actually tested this by running
> make dt_binding_check.
> 
> If I intentionally break the example by setting a wrong compatible
> string, I get a "False schema does not allow" type message, same
> as when I use the syscon-pllreset property with the wrong compat
> string.
> 
> Is the existing check broken too? Is there something different
> supposed to happen?

I did not say it was broken. It is functional. It is just not minimized.

Rob