On Wed, Feb 26, 2025 at 02:09:22PM +0100, Thierry Bultel wrote: > Document RZ/T2H (a.k.a r9a09g077) in SCI binding. > > Signed-off-by: Thierry Bultel <thierry.bultel.yh@xxxxxxxxxxxxxx> > Reviewed-by: Paul Barker <paul.barker.ct@xxxxxxxxxxxxxx> > --- > .../bindings/serial/renesas,sci.yaml | 64 ++++++++++++------- > 1 file changed, 40 insertions(+), 24 deletions(-) > > diff --git a/Documentation/devicetree/bindings/serial/renesas,sci.yaml b/Documentation/devicetree/bindings/serial/renesas,sci.yaml > index 64d3db6e54e5..2c4080283963 100644 > --- a/Documentation/devicetree/bindings/serial/renesas,sci.yaml > +++ b/Documentation/devicetree/bindings/serial/renesas,sci.yaml > @@ -9,9 +9,6 @@ title: Renesas Serial Communication Interface > maintainers: > - Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > > -allOf: > - - $ref: serial.yaml# > - > properties: > compatible: > oneOf: > @@ -22,6 +19,8 @@ properties: > - renesas,r9a07g054-sci # RZ/V2L > - const: renesas,sci # generic SCI compatible UART > > + - const: renesas,r9a09g077-sci # RZ/T2H > + Perhaps explain in the commit msg why the 'renesas,sci' is not applicable for this chip. > - items: > - const: renesas,sci # generic SCI compatible UART > > @@ -54,8 +53,6 @@ properties: > - fck # UART functional clock > - sck # optional external clock input > > - uart-has-rtscts: false > - > required: > - compatible > - reg > @@ -63,25 +60,44 @@ required: > - clocks > - clock-names > > -if: > - properties: > - compatible: > - contains: > - enum: > - - renesas,r9a07g043-sci > - - renesas,r9a07g044-sci > - - renesas,r9a07g054-sci > -then: > - properties: > - resets: > - maxItems: 1 > - > - power-domains: > - maxItems: 1 > - > - required: > - - resets > - - power-domains > +allOf: > + - $ref: serial.yaml# > + - if: > + properties: > + compatible: > + contains: > + const: renesas,r9a09g077-sci > + then: > + properties: > + uart-has-rtscts: true Drop. It is already allowed. > + power-domains: > + maxItems: 1 > + required: > + - power-domains > + > + else: > + properties: > + uart-has-rtscts: false > + > + - if: > + properties: > + compatible: > + contains: > + enum: > + - renesas,r9a07g043-sci > + - renesas,r9a07g044-sci > + - renesas,r9a07g054-sci > + then: > + properties: > + resets: > + maxItems: 1 > + > + power-domains: > + maxItems: 1 > + > + required: > + - resets > + - power-domains > > unevaluatedProperties: false > > -- > 2.43.0 >
