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. > + - loongson,ls3a5000-uart > + - loongson,ls2k2000-uart > + - items: > + - enum: > + - loongson,ls2k1000-uart > + - loongson,ls2k0500-uart > + - const: loongson,ls7a-uart Just use real SoC names. > + - items: > + - enum: > + - loongson,ls2k1500-uart > + - const: loongson,ls2k2000-uart > + - items: > + - enum: > + - loongson,ls3a6000-uart > + - const: loongson,ls3a5000-uart Best regards, Krzysztof