Re: [PATCH 01/10] dt-bindings: display: bridge: it66121: Add compatible string for IT6610

From: Paul Cercueil
Date: Fri Dec 16 2022 - 07:22:14 EST


Hi Krzysztof,

Le vendredi 16 décembre 2022 à 12:21 +0100, Krzysztof Kozlowski a
écrit :
> On 16/12/2022 11:46, Paul Cercueil wrote:
>
> > > >  properties:
> > > >    compatible:
> > > > -    const: ite,it66121
> > > > +    enum:
> > > > +      - ite,it66121
> > > > +      - ite,it6610
>
> These should be ordered alphabetically. What's with the tendency of
> adding always to the end?

I'm too used to the "inverse christmas tree" sort :)

I can send a quickfix patch if you really want alphabetical order.

> > > >
> > > >    reg:
> > > >      maxItems: 1
> > > > --
> > > > 2.35.1
> > > >
> > >
> > > Reviewed-by: Robert Foss <robert.foss@xxxxxxxxxx>
> >
> > Series applied to drm-misc-next.
> >
> I wished you give DT maintainers a bit more time than two days.

Noted. Sorry about that.

Cheers,
-Paul