On Tue, Oct 24, 2017 at 12:07 PM, Rob Herring <robh@xxxxxxxxxx> wrote: > On Wed, Oct 18, 2017 at 10:01:36AM -0700, Andrey Smirnov wrote: > > Commit msg? > OK, will add in v9. >> Cc: linux-kernel@xxxxxxxxxxxxxxx >> Cc: devicetree@xxxxxxxxxxxxxxx >> Cc: linux-watchdog@xxxxxxxxxxxxxxx >> Cc: cphealy@xxxxxxxxx >> Cc: Lucas Stach <l.stach@xxxxxxxxxxxxxx> >> Cc: Nikita Yushchenko <nikita.yoush@xxxxxxxxxxxxxxxxxx> >> Cc: Lee Jones <lee.jones@xxxxxxxxxx> >> Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> >> Cc: Pavel Machek <pavel@xxxxxx> >> Cc: Andy Shevchenko <andy.shevchenko@xxxxxxxxx> >> Cc: Guenter Roeck <linux@xxxxxxxxxxxx> >> Cc: Rob Herring <robh@xxxxxxxxxx> >> Cc: Johan Hovold <johan@xxxxxxxxxx> >> Cc: Mark Rutland <mark.rutland@xxxxxxx> >> Signed-off-by: Nikita Yushchenko <nikita.yoush@xxxxxxxxxxxxxxxxxx> >> Signed-off-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx> >> --- >> Documentation/devicetree/bindings/watchdog/zii,rave-sp-wdt.txt | 10 ++++++++++ >> 1 file changed, 10 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/watchdog/zii,rave-sp-wdt.txt >> >> diff --git a/Documentation/devicetree/bindings/watchdog/zii,rave-sp-wdt.txt b/Documentation/devicetree/bindings/watchdog/zii,rave-sp-wdt.txt >> new file mode 100644 >> index 000000000000..f242c3eed0dc >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/watchdog/zii,rave-sp-wdt.txt >> @@ -0,0 +1,10 @@ >> +Zodiac Inflight Innovations RAVE Supervisory Processor Watchdog >> + >> +Required properties: >> + >> + - compatible: Should be "zii,rave-sp-watchdog" >> + > > Must be a child of ? defined in ? Good point, will add. > >> +Optional properties: >> + >> + - Two byte nvmem cell "wdt_timeout", specified as per >> + Documentation/devicetree/bindings/nvmem/nvmem.txt > > prop name: definition... is the normal format. > > Also, don't use '_' in property names. > OK, will fix. Thanks, Andrey Smirnov -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
