Hi, thank you for review! Am 05.11.2015 um 21:43 schrieb Rob Herring: > On Thu, Nov 05, 2015 at 10:06:56AM +0100, Oleksij Rempel wrote: >> Add WD support for Alphascale asm9260 SoC. This driver >> provide support for different function modes: >> - HW mode to trigger SoC reset on timeout >> - SW mode do soft reset if needed >> - DEBUG mode >> >> Optional support for stopping watchdog. If reset binding are not provided >> this driver will work in nowayout mode. >> >> Signed-off-by: Oleksij Rempel <linux@xxxxxxxxxxxxxxxx> >> --- >> .../bindings/watchdog/alphascale-asm9260.txt | 39 ++ > > It is preferred that bindings are a separate patch. > >> drivers/watchdog/Kconfig | 9 + >> drivers/watchdog/Makefile | 1 + >> drivers/watchdog/asm9260_wdt.c | 415 +++++++++++++++++++++ >> 4 files changed, 464 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/watchdog/alphascale-asm9260.txt >> create mode 100644 drivers/watchdog/asm9260_wdt.c >> >> diff --git a/Documentation/devicetree/bindings/watchdog/alphascale-asm9260.txt b/Documentation/devicetree/bindings/watchdog/alphascale-asm9260.txt >> new file mode 100644 >> index 0000000..6e54d1f >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/watchdog/alphascale-asm9260.txt >> @@ -0,0 +1,39 @@ >> +Alphascale asm9260 Watchdog timer >> + >> +Required properties: >> + >> +- compatible : should be "alphascale,asm9260-wdt". >> +- reg : Specifies base physical address and size of the registers. >> +- clocks : the clocks feeding the watchdog timer. See clock-bindings.txt >> +- clock-names : should be set to >> + "mod" - source for tick counter. >> + "ahb" - ahb gate. >> + >> +Optional properties: >> +- resets : phandle pointing to the system reset controller with correct >> + reset line index for watchdog controller reset. This propertie is > > s/propertie/property/ > >> + required if you need to disable "nowayout" and it neened only with >> + CONFIG_WATCHDOG_NOWAYOUT=n. > > The DT cannot depend on certain kernel configs. > >> + Without reseting this WD controller, it is not possible to stop > > s/reseting/resetting/ > >> + counter. >> +- reset-names : should be set to "wdt_rst" if "resets" is used. >> +- timeout-sec : shall contain the default watchdog timeout in seconds, >> + if unset, the default timeout is 30 seconds. >> +- alphascale,mode : tree modes are supported > > s/tree/three/ > >> + "hw" - hw reset (defaul). > > s/defaul/default/ > >> + "sw" - sw reset. >> + "debug" - no action is taken. > > Seems like this should be a generic wdog property. > what do you mean? Using "mode" instead of "alphascale,mode" and implement it in WD framework? -- Regards, Oleksij
Attachment:
signature.asc
Description: OpenPGP digital signature