On Fri, Jun 12, 2020 at 04:19:03PM +0200, Thierry Reding wrote: > From: Thierry Reding <treding@xxxxxxxxxx> > > On Tegra the UART is described using additional properties, such as > clock-names, reset-names, dmas and dma-names. Document them in the > bindings so that Tegra device trees are properly validated. > > Signed-off-by: Thierry Reding <treding@xxxxxxxxxx> > --- > .../devicetree/bindings/serial/8250.yaml | 26 +++++++++++++++++++ > 1 file changed, 26 insertions(+) > > diff --git a/Documentation/devicetree/bindings/serial/8250.yaml b/Documentation/devicetree/bindings/serial/8250.yaml > index c1d4c196f005..9c8fad27c3f3 100644 > --- a/Documentation/devicetree/bindings/serial/8250.yaml > +++ b/Documentation/devicetree/bindings/serial/8250.yaml > @@ -28,6 +28,32 @@ allOf: > const: 2 > required: > - reg-shift > + - if: > + properties: > + compatible: > + contains: > + enum: > + - nvidia,tegra20-uart Can use 'const' here instead. > + then: > + properties: > + clock-names: > + $ref: "/schemas/types.yaml#/definitions/string-array" Don't need type. > + items: > + - const: serial > + > + dmas: > + $ref: "/schemas/types.yaml#/definitions/phandle-array" How many? > + > + dma-names: > + $ref: "/schemas/types.yaml#/definitions/string-array" > + items: > + - const: rx > + - const: tx > + > + reset-names: > + $ref: "/schemas/types.yaml#/definitions/string-array" > + items: > + - const: serial > - if: > not: > properties: > -- > 2.24.1 >
