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

From: Ian Lartey
Date: Wed Feb 27 2013 - 17:29:17 EST


On 27/02/13 20:36, Stephen Warren wrote:
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@xxxxxxxxxxxxxxx>

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.

I've not reviewed your comments on the other patches, so I'll look at
those before responding properly.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/