Messages in this thread |  | | Date | Mon, 18 Nov 2013 17:26:06 +0200 | From | "ivan.khoronzhuk" <> | Subject | Re: Fwd: [PATCH 7/8] watchdog: davinci: add "clocks" property |
| |
On 11/17/2013 04:28 AM, Guenter Roeck wrote: > On 11/06/2013 03:32 AM, ivan.khoronzhuk wrote: >> The Keystone arch is using clocks in DT and source clock for watchdog >> has to be specified, so add this to binding. >> >> Signed-off-by: Ivan Khoronzhuk <ivan.khoronzhuk@ti.com> >> --- >> .../devicetree/bindings/watchdog/davinci-wdt.txt | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git >> a/Documentation/devicetree/bindings/watchdog/davinci-wdt.txt >> b/Documentation/devicetree/bindings/watchdog/davinci-wdt.txt >> index fddced9..4db4d0e 100644 >> --- a/Documentation/devicetree/bindings/watchdog/davinci-wdt.txt >> +++ b/Documentation/devicetree/bindings/watchdog/davinci-wdt.txt >> @@ -7,6 +7,10 @@ Required properties: >> >> - reg : Should contain WDT registers location and length >> >> +- clocks: phandle reference to the controller clock. >> + Required only for Keystone arch. >> + See clock-bindings.txt >> + > > Yet another form of formatting. Also, wonder if it makes sense to merge > this with the patch adding keystone support. >
Ok, I'll squash them.
>> Optional properties: >> >> - timeout-sec: Contains the watchdog timeout in seconds >> @@ -21,4 +25,5 @@ wdt: wdt@2320000 { >> compatible = "ti,davinci-wdt"; >> reg = <0x02320000 0x80>; >> timeout-sec = <30>; >> + clocks = <&clkwdtimer0>; >> }; >> >
-- Regards, Ivan Khoronzhuk
|  |