On Wed, Sep 17, 2014 at 10:58 PM, Beniamino Galvani <b.galvani@xxxxxxxxx> wrote: > On Wed, Sep 17, 2014 at 10:12:03PM +0200, Carlo Caione wrote: >> Signed-off-by: Carlo Caione <carlo@xxxxxxxxxx> >> --- >> Documentation/devicetree/bindings/watchdog/meson-wdt.txt | 13 +++++++++++++ >> 1 file changed, 13 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/watchdog/meson-wdt.txt >> >> diff --git a/Documentation/devicetree/bindings/watchdog/meson-wdt.txt b/Documentation/devicetree/bindings/watchdog/meson-wdt.txt >> new file mode 100644 >> index 0000000..e66c0c3 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/watchdog/meson-wdt.txt >> @@ -0,0 +1,13 @@ >> +Meson SoCs Watchdog timer >> + >> +Required properties: >> + >> +- compatible : should be "amlogic,meson-wdt" > > Hi Carlo, > > the compatible property should describe a specific version of the > device, so maybe "amlogic,meson6-wdt" would be more appropriate. Agree. Fix in v3. Thanks, -- Carlo Caione -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html