lkml.org 
[lkml]   [2013]   [Nov]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: 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


    \
     
     \ /
      Last update: 2013-11-18 16:41    [W:4.410 / U:0.500 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site