Skip to contentSkip to conversation

dt-bindings: serial: snps-dw-apb-uart: Add Synaptics sl2610 uart

PATCH New Jisheng Zhang 2 participants 2 messages Oct 14

Subsystems (2)
Device tree bindings TTY layer and serial drivers
lore
Share & tools
lore

Jisheng Zhang

author
#0 lore raw

PATCH v3 1/1 dt-bindings: serial: snps-dw-apb-uart: Add Synaptics sl2610 uart

Add the compatible for the UARTs of the Synaptics SL2610, which are compatible with the Synopsys DesignWare APB UART.

Signed-off-by Jisheng Zhang <[email protected]>
Acked-by Conor Dooley <[email protected]>

---

since v2:
- collect Acked-by tag
- rebase on the latest tty-testing

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(-)
Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml +11 -1
index 86dc25919311..e2ca126d58ba 100644
@@ -34,7 +34,16 @@ allOf:
       properties:
         resets:
           minItems: 2
-    else:
+
+  - if:
+      not:
+        properties:
+          compatible:
+            contains:
+              enum:
+                - starfive,jh7110-uart
+                - syna,sl2610-uart
+    then:
       properties:
         resets:
           maxItems: 1

@@ -80,6 +89,7 @@ properties:
               - starfive,jh7100-hsuart
               - starfive,jh7100-uart
               - starfive,jh7110-uart
+              - syna,sl2610-uart
               - ti,tda54-uart
               - ultrarisc,dp1000-uart
           - const: snps,dw-apb-uart
Signature · 1 line
2.53.0
AI review
#1 lore raw
Sashiko has reviewed this patch and found no issues. It looks great!
Quoted 4 lines

Add the compatible for the UARTs of the Synaptics SL2610, which are compatible with the Synopsys DesignWare APB UART.

Signed-off-by Jisheng Zhang <[email protected]>

Sashiko has reviewed this patch and found no issues. It looks great!

Signature · 1 line
Sashiko AI review · https://sashiko.dev/#/patchset/[email protected]?part=1