| Message ID | 20220321154232.56315-4-phil.edworthy@renesas.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Add new Renesas RZ/V2M SoC and Renesas RZ/V2M EVK support | expand |
On 21/03/2022 16:42, Phil Edworthy wrote: > The Renesas RZ/V2M (r9a09g011) SoC uses a uart that is compatible with the > EMMA Mobile SoC. > > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com> > Reviewed-by: Biju Das <biju.das.jz@bp.renesas.com> > --- > .../devicetree/bindings/serial/renesas,em-uart.yaml | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/serial/renesas,em-uart.yaml b/Documentation/devicetree/bindings/serial/renesas,em-uart.yaml > index e98ec48fee46..42733eaa0ece 100644 > --- a/Documentation/devicetree/bindings/serial/renesas,em-uart.yaml > +++ b/Documentation/devicetree/bindings/serial/renesas,em-uart.yaml > @@ -14,7 +14,11 @@ allOf: > > properties: > compatible: > - const: renesas,em-uart > + oneOf: No need for oneOf, you have just one element below. > + - items: > + - enum: > + - renesas,r9a09g011-uart # RZ/V2M > + - const: renesas,em-uart # generic EMMA Mobile compatible UART > Does not look like you tested it... Best regards, Krzysztof
Hi Krzysztof, On 23 March 2022 10:42, Krzysztof Kozlowski wrote: > On 21/03/2022 16:42, Phil Edworthy wrote: > > The Renesas RZ/V2M (r9a09g011) SoC uses a uart that is compatible with > the > > EMMA Mobile SoC. > > > > Signed-off-by: Phil Edworthy <phil.edworthy@renesas.com> > > Reviewed-by: Biju Das <biju.das.jz@bp.renesas.com> > > --- > > .../devicetree/bindings/serial/renesas,em-uart.yaml | 6 +++++- > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > diff --git a/Documentation/devicetree/bindings/serial/renesas,em- > uart.yaml b/Documentation/devicetree/bindings/serial/renesas,em-uart.yaml > > index e98ec48fee46..42733eaa0ece 100644 > > --- a/Documentation/devicetree/bindings/serial/renesas,em-uart.yaml > > +++ b/Documentation/devicetree/bindings/serial/renesas,em-uart.yaml > > @@ -14,7 +14,11 @@ allOf: > > > > properties: > > compatible: > > - const: renesas,em-uart > > + oneOf: > > No need for oneOf, you have just one element below. Ok > > + - items: > > + - enum: > > + - renesas,r9a09g011-uart # RZ/V2M > > + - const: renesas,em-uart # generic EMMA Mobile > compatible UART > > > > Does not look like you tested it... Sorry about that. > > Best regards, > Krzysztof Thanks Phil
diff --git a/Documentation/devicetree/bindings/serial/renesas,em-uart.yaml b/Documentation/devicetree/bindings/serial/renesas,em-uart.yaml index e98ec48fee46..42733eaa0ece 100644 --- a/Documentation/devicetree/bindings/serial/renesas,em-uart.yaml +++ b/Documentation/devicetree/bindings/serial/renesas,em-uart.yaml @@ -14,7 +14,11 @@ allOf: properties: compatible: - const: renesas,em-uart + oneOf: + - items: + - enum: + - renesas,r9a09g011-uart # RZ/V2M + - const: renesas,em-uart # generic EMMA Mobile compatible UART reg: maxItems: 1