On 07/03/2024 00:10, Prabhakar wrote: > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > > In preparation for adding more validation checks move the ref for > 'serial.yaml' to the end and also move reset check in 'allOf' block. > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > --- > .../bindings/serial/renesas,scif.yaml | 30 +++++++++---------- > 1 file changed, 15 insertions(+), 15 deletions(-) > > diff --git a/Documentation/devicetree/bindings/serial/renesas,scif.yaml b/Documentation/devicetree/bindings/serial/renesas,scif.yaml > index 4610a5bd580c..af72c3420453 100644 > --- a/Documentation/devicetree/bindings/serial/renesas,scif.yaml > +++ b/Documentation/devicetree/bindings/serial/renesas,scif.yaml > @@ -9,9 +9,6 @@ title: Renesas Serial Communication Interface with FIFO (SCIF) > maintainers: > - Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > > -allOf: > - - $ref: serial.yaml# > - > properties: > compatible: > oneOf: > @@ -160,18 +157,21 @@ required: > - clock-names > - power-domains > > -if: > - properties: > - compatible: > - contains: > - enum: > - - renesas,rcar-gen2-scif > - - renesas,rcar-gen3-scif > - - renesas,rcar-gen4-scif > - - renesas,scif-r9a07g044 > -then: > - required: > - - resets > +allOf: > + - $ref: serial.yaml# > + Yeah, that's why usually we ask to put '$ref' and 'if' in allOf: block - saves you one syntax reindent which will confuse git blame. Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> Best regards, Krzysztof
