On 20/01/2025 16:46, Benjamin Larsson wrote: >>>>> diff --git a/Documentation/devicetree/bindings/serial/8250.yaml b/Documentation/devicetree/bindings/serial/8250.yaml >>>>> index 692aa05500fd..2fbb972e5460 100644 >>>>> --- a/Documentation/devicetree/bindings/serial/8250.yaml >>>>> +++ b/Documentation/devicetree/bindings/serial/8250.yaml >>>>> @@ -63,6 +63,8 @@ properties: >>>>> - const: mrvl,pxa-uart >>>>> - const: nuvoton,wpcm450-uart >>>>> - const: nuvoton,npcm750-uart >>>>> + - const: airoha,airoha-uart >>>>> + - const: airoha,airoha-hsuart >>>> I assume you placed it matching existing order (kind of alphabetical for >>>> compatibles with vendors)? >>>> >>>> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> >>>> >>>> Best regards, >>>> Krzysztof >>>> >>> Hi, I placed it after nuvoton that was the most recent addition to the >>> serial8250_config uart_config[] table that was also added to the >> But wasn't nuvoton placed in specific, ordered place? > > Well not really as wpcm450-uart is placed before npcm750-uart in Uh, indeed. :/ > 8250.yaml. But I will move the entries to below ns16850 in the next > revision to keep the vendor list in alphabetical order. Best regards, Krzysztof