在 2024/8/26 13:59, Krzysztof Kozlowski 写道:
On Mon, Aug 26, 2024 at 10:47:03AM +0800,zhenghaowei@xxxxxxxxxxx wrote:
From: Haowei Zheng<zhenghaowei@xxxxxxxxxxx>
Add Loongson UART controller binding with DT schema format using
json-schema.
Signed-off-by: Haowei Zheng<zhenghaowei@xxxxxxxxxxx>
---
.../bindings/serial/loongson,uart.yaml | 63 +++++++++++++++++++
1 file changed, 63 insertions(+)
create mode 100644 Documentation/devicetree/bindings/serial/loongson,uart.yaml
Changes in V2:
- Correct the schema formatting errors.
- file name changed from 'loongson-uart.yaml' to 'loongson,ls7a-uart.yaml'
- Replace 'loongson,loongson-uart' with 'loongson,ls7a-uart'.
Changes in V3:
- Change the filename from 'loongson,ls7a-uart.yaml' to 'loongson,uart.yaml'.
- Drop newly defined features: fractional-division, rts-invert, dtr-invert,
cts-invert and dsr-invert.
- Add three specific SoC: 'loongson,ls7a-uart', 'loongson,ls3a5000-uart' and
'loongson,ls2k2000-uart'.
- Drop 'LOONGSON UART DRIVER' description in MAINTAINERS.
diff --git a/Documentation/devicetree/bindings/serial/loongson,uart.yaml b/Documentation/devicetree/bindings/serial/loongson,uart.yaml
new file mode 100644
index 000000000000..19a65dd5be9f
--- /dev/null
+++ b/Documentation/devicetree/bindings/serial/loongson,uart.yaml
@@ -0,0 +1,63 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id:http://devicetree.org/schemas/loongson,uart.yaml#
+$schema:http://devicetree.org/meta-schemas/core.yaml#
+
+title: Loongson UART
+
+maintainers:
+ - Haowei Zheng<zhenghaowei@xxxxxxxxxxx>
+
+allOf:
+ - $ref: serial.yaml
+
+properties:
+ compatible:
+ oneOf:
+ - enum:
+ - loongson,ls7a-uart
Quick look tells me there is no such soc like ls7a. If there is such,
please point me to the DTSI.
I had a problem with my description of ls7a, it's just a bridge chip, but
both ls2k0500 and ls2k1000 share the same UART controller as it.
+ - loongson,ls3a5000-uart
+ - loongson,ls2k2000-uart
+ - items:
+ - enum:
+ - loongson,ls2k1000-uart
+ - loongson,ls2k0500-uart
+ - const: loongson,ls7a-uart
Just use real SoC names.
The ls7a is not an SoC; it is typically used in conjunction with
Loongson 3 series
processors and boots via ACPI. Currently, there is no corresponding DTSI
provided.
+ - items:
+ - enum:
+ - loongson,ls2k1500-uart
+ - const: loongson,ls2k2000-uart
+ - items:
+ - enum:
+ - loongson,ls3a6000-uart
+ - const: loongson,ls3a5000-uart
Best regards,
Krzysztof