Re: [PATCH 1/2] dt-bindings: serial: ingenic: Add the X1600

[Krzysztof Kozlowski]

On Sat, Oct 24, 2026 at 07:15:14AM +0200, Olivier Louvignes wrote:
> The X1600's UARTs have the X1000's registers as far as the 8250 core
> uses them, but the driver has to tell them apart to set their bit clock
> count. Add ingenic,x1600-uart, with the X1000's as fallback.
> 
> Signed-off-by: Olivier Louvignes <olivier@xxxxxxxxx>
> ---
>  Documentation/devicetree/bindings/serial/ingenic,uart.yaml | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/serial/ingenic,uart.yaml b/Documentation/devicetree/bindings/serial/ingenic,uart.yaml
> index d5f153bdeb0d..635342d43ddb 100644
> --- a/Documentation/devicetree/bindings/serial/ingenic,uart.yaml
> +++ b/Documentation/devicetree/bindings/serial/ingenic,uart.yaml
> @@ -32,6 +32,9 @@ properties:
>        - items:
>            - const: ingenic,jz4725b-uart
>            - const: ingenic,jz4740-uart
> +      - items:
> +          - const: ingenic,x1600-uart
> +          - const: ingenic,x1000-uart

These should be sorted by fallback, so list using x1000 should be the
last (x > j).

With this fixed:

Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxxxxx>

Best regards,
Krzysztof