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

[Conor Dooley]

On Thu, Oct 08, 2026 at 09:49:07PM +0800, Jisheng Zhang wrote:
> 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.

In the future, please mention this so it doesn't look like an oversight.
Acked-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
pw-bot: not-applicable

Thanks,
Conor.

> 
> > 
> > > +    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
> > > 
> 
>

Attachment: signature.asc
Description: PGP signature