On 07/06/2022 15:56, Phil Edworthy wrote: > Add the documentation for the r9a09g011 SoC, but in doing so also > reorganise the doc to make it easier to read. > Additionally, make the binding require an interrupt to be specified. > Whilst the driver does not need an interrupt, all of the SoCs that use > this binding actually provide one. > > Signed-off-by: Phil Edworthy <phil.edworthy@xxxxxxxxxxx> > Reviewed-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> > --- > .../bindings/watchdog/renesas,wdt.yaml | 63 ++++++++++++------- > 1 file changed, 42 insertions(+), 21 deletions(-) > > diff --git a/Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml b/Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml > index a8d7dde5271b..6473734921e3 100644 > --- a/Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml > +++ b/Documentation/devicetree/bindings/watchdog/renesas,wdt.yaml > @@ -31,6 +31,11 @@ properties: > - renesas,r9a07g054-wdt # RZ/V2L > - const: renesas,rzg2l-wdt > > + - items: > + - enum: > + - renesas,r9a09g011-wdt # RZ/V2M > + - const: renesas,rzv2m-wdt # RZ/V2M > + > - items: > - enum: > - renesas,r8a7742-wdt # RZ/G1H > @@ -70,13 +75,27 @@ properties: > reg: > maxItems: 1 > > - interrupts: true > + interrupts: > + minItems: 1 > + items: > + - description: Timeout > + - description: Parity error > > - interrupt-names: true > + interrupt-names: This also needs minItems > + items: > + - const: wdt > + - const: perrout > > - clocks: true > + clocks: > + minItems: 1 > + items: > + - description: Register access clock > + - description: Main clock > > - clock-names: true > + clock-names: Ditto > + items: > + - const: pclk > + - const: oscclk > Best regards, Krzysztof
