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

From: Krzysztof Kozlowski
Date: Fri Dec 16 2022 - 08:22:14 EST


On 16/12/2022 13:21, Paul Cercueil wrote:
> 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 :)

Since these are not variables and they will not get shorter, any
christmas tree sorting here is the same conflict-prone as adding to the end.

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

No, no need.


Best regards,
Krzysztof