Re: [PATCH v5 1/7] dt-bindings: display: verisilicon,dc: generalize for single-output variants

[Conor Dooley <conor@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1099 bytes --]

On Fri, Jun 26, 2026 at 05:33:26PM +0800, Icenowy Zheng wrote:
> > > > 
> > > > > > > +
> > > > > > > +        resets:
> > > > > > > +          minItems: 1
> > > > > > > +          maxItems: 1
> > > > > > > +
> > > > > > > +        reset-names:
> > > > > > > +          items:
> > > > > > > +            - const: core
> > > > > > 
> > > > > > This is just maxItems: 1.
> > > > > 
> > > > > Well the implicit rules of DT binding schemas are quite
> > > > > weird...
> > > > 
> > > > I don't think it is that strange, as the binding has
> > > >   reset-names:
> > > >     items:
> > > >       - const: core
> > > >       - const: axi
> > > >       - const: ahb
> > > 
> > > Ah does the list constraint the order of items? If it constrains
> > > the
> > 
> > It does, yes.
> > Alternatively, using an enum permits free ordering.
> 
> Ah in this case this should be converted to an enum, I think.
> 
> Should I send a patch for converting it?

Why do you think it should be an enum? We don't currently have any users
of this that only provide no core or no axi reset.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]