Re: [PATCH v2] dt-bindings: serial: snps-dw-apb-uart: Add Synaptics sl2610 uart

[Jisheng Zhang]

On Wed, Oct 07, 2026 at 02:04:05PM +0100, Conor Dooley wrote:
> On Mon, Oct 05, 2026 at 11:19:18PM +0800, Jisheng Zhang wrote:
> > Add the compatible for the UARTs of the Synaptics SL2610, which are
> > compatible with the Synopsys DesignWare APB UART.
> > 
> > Signed-off-by: Jisheng Zhang <jszhang@xxxxxxxxxx>
> > ---
> > 
> > since v1:
> >  - use sl2610 instead of sl261x
> >  - send as a separate patch
> > 
> >  .../devicetree/bindings/serial/snps-dw-apb-uart.yaml | 12 +++++++++++-
> >  1 file changed, 11 insertions(+), 1 deletion(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml b/Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml
> > index 664305de4986..a40f2737e71b 100644
> > --- a/Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml
> > +++ b/Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml
> > @@ -34,7 +34,16 @@ allOf:
> >        properties:
> >          resets:
> >            minItems: 2
> > -    else:
> > +
> > +  - if:
> > +      not:
> > +        properties:
> > +          compatible:
> > +            contains:
> > +              enum:
> > +                - starfive,jh7110-uart
> > +                - syna,sl2610-uart
> 
> Is the number of uart resets on this device variable?

Yes, some uarts have only 1 reset, while others have two resets.

> 
> > +    then:
> >        properties:
> >          resets:
> >            maxItems: 1
> > @@ -80,6 +89,7 @@ properties:
> >                - starfive,jh7100-hsuart
> >                - starfive,jh7100-uart
> >                - starfive,jh7110-uart
> > +              - syna,sl2610-uart
> >                - ultrarisc,dp1000-uart
> >            - const: snps,dw-apb-uart
> >        - const: snps,dw-apb-uart
> > -- 
> > 2.53.0
> >