Hi Fabrizio, On Thu, Jan 25, 2018 at 7:02 PM, Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> wrote: > This commit documents the compatibility with R-Car Gen2 and RZ/G > devices by defining the generifc compatible string "renesas,rcar-gen2-wdt". generic + SoC-specific compatible values. > Signed-off-by: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> > Signed-off-by: Ramesh Shanmugasundaram <ramesh.shanmugasundaram@xxxxxxxxxxxxxx> > --- > Documentation/devicetree/bindings/watchdog/renesas-wdt.txt | 13 ++++++++----- > 1 file changed, 8 insertions(+), 5 deletions(-) > > diff --git a/Documentation/devicetree/bindings/watchdog/renesas-wdt.txt b/Documentation/devicetree/bindings/watchdog/renesas-wdt.txt > index bf6d1ca..436dca5 100644 > --- a/Documentation/devicetree/bindings/watchdog/renesas-wdt.txt > +++ b/Documentation/devicetree/bindings/watchdog/renesas-wdt.txt > @@ -1,17 +1,20 @@ > Renesas Watchdog Timer (WDT) Controller > > Required properties: > -- compatible : Should be "renesas,<soctype>-wdt", and > - "renesas,rcar-gen3-wdt" or "renesas,rza-wdt" as fallback. > + - compatible : Must be "renesas,<soctype>-wdt", followed by a generic > + fallback compatible string when compatible with the generic > + version. > Examples with soctypes are: > + - "renesas,r8a7743-wdt" (RZ/G1M) > + - "renesas,r8a7745-wdt" (RZ/G1E) I guess you wanted to add r8a779[0-4], too, as you use (some of) them in later patches in this series? > - "renesas,r8a7795-wdt" (R-Car H3) > - "renesas,r8a7796-wdt" (R-Car M3-W) > - "renesas,r8a77995-wdt" (R-Car D3) > - "renesas,r7s72100-wdt" (RZ/A1) > + The generic compatible string must be: > + - "renesas,rcar-gen2-wdt" for R-Car Gen2 and RZ/G > + - "renesas,rcar-gen3-wdt" for R-Car Gen3 > > - When compatible with the generic version, nodes must list the SoC-specific > - version corresponding to the platform first, followed by the generic > - version. > > - reg : Should contain WDT registers location and length > - clocks : the clock feeding the watchdog timer. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds