Re: Fwd: [PATCH 7/8] watchdog: davinci: add "clocks" property

["ivan.khoronzhuk" <ivan.khoronzhuk@ti.com>]

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