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

[David Oberhollenzer]

Hi,

On 8/5/26 9:39 AM, Krzysztof Kozlowski wrote:
> Not a valid naming, missing vendor prefix and syscon looks like
> Linuxism, since you called it "system control module".

I agree that this is a very vendor specific kluge and I suspected
it might need a vendor prefix. However, I ultimately tried to stick
to existing conventions.

There are 3 more syscon properties defined for this device:
 - syscon-phy-power
 - syscon-pllreset
 - syscon-pcs

None of them have a vendor prefix, all of them use the
description "system control module".

I take it, the existing properties already break best practices
that were established later? Or is there another reason they
lack a vendor prefix?

> > +    $ref: /schemas/types.yaml#/definitions/phandle-array
> > +    maxItems: 1
> > +    items:
> > +      items:
>
> Missing -
Likewise, I simply copied from the syscon blocks and made some
minor changes. Hence also the identical description. So this is
broken for the existing properties as well?

Thanks,

David