Re: [PATCH 1/2] watchdog: add Palmas Watchdog support

[Stephen Warren <swarren@wwwdotorg.org>]

On 02/27/2013 01:17 PM, Ian Lartey wrote:
> On 27/02/13 18:38, Stephen Warren wrote:
>> On 02/27/2013 11:36 AM, Ian Lartey wrote:
>>> From: Graeme Gregory <gg@slimlogic.co.uk>
>>>
>>> Add support for the Palmas watchdog timer which has a timeout
>>> configurable
>>> from 1s to 128s.
>>
>>> diff --git a/drivers/watchdog/palmas_wdt.c
>>> b/drivers/watchdog/palmas_wdt.c
>>
>>> +static struct of_device_id of_palmas_match_tbl[] = {
>>> +    { .compatible = "ti,palmas-wdt", },
>>> +    { /* end */ }
>>> +};
>>
>> Is there a DT binding document for that compatible value?
> 
> The device tree for this compatible value is actually empty,
> shall I remove it or is it OK to show it as empty in the
> bindings document ?
> There will be other palmas device bindings in this document.

Well, you need a binding document that describes how to create this
node. If it needs no properties other than compatible, it'll be a simple
document!

Did you review my comments on the other patches re: whether the
sub-modules of this HW actually should be sub-nodes in DT or not? If
not, then you don't need this node in DT, and hence there will be no
compatible value for the node, and hence no need to write a binding
document.