Re: [PATCH 4/7] watchdog: s3c2410: Add support for device tree based probe

Hi Grant,

On 17 February 2011 05:47, Grant Likely [off-list ref] wrote:

On Sat, Feb 12, 2011 at 06:17:02PM +0530, Thomas Abraham wrote:

quoted

This patch adds the of_match_table to enable s3c2410-wdt driver
to be probed when watchdog device node is found in the device tree.

Signed-off-by: Thomas Abraham <redacted>
---
 drivers/watchdog/s3c2410_wdt.c |   10 ++++++++++
 1 files changed, 10 insertions(+), 0 deletions(-)

Need to add documentation for compatible = "samsung,s3c2410-wdt" to
Documentation/devicetree/bindings/wdt before this patch can be merged.

A couple more comments below, but I'm okay with this one.  I'd go
ahead and submit it to the linux-watchdog-u79uwXL29TY76Z2rM5mHXA@public.gmane.org list and cc:
Wim.  It can be mainlined right away after fixing up the comments.
I'll ack it when you repost and then work out with Wim if it should be
merged through his tree or through mine.

quoted

diff --git a/drivers/watchdog/s3c2410_wdt.c b/drivers/watchdog/s3c2410_wdt.c
index ae53662..a9edd50 100644
--- a/drivers/watchdog/s3c2410_wdt.c
+++ b/drivers/watchdog/s3c2410_wdt.c

@@ -592,6 +592,13 @@ static int s3c2410wdt_resume(struct platform_device *dev)

 #define s3c2410wdt_resume  NULL
 #endif /* CONFIG_PM */

+#ifdef CONFIG_OF
+static const struct of_device_id s3c2410_wdt_match[] = {
+     { .compatible = "samsung,s3c2410-wdt" },
+     {},
+};
+MODULE_DEVICE_TABLE(of, s3c2410_wdt_match);
+#endif

#else
#define s3c2410_wdt_match NULL
#endif

quoted

 static struct platform_driver s3c2410wdt_driver = {
      .probe          = s3c2410wdt_probe,

@@ -602,6 +609,9 @@ static struct platform_driver s3c2410wdt_driver = {

      .driver         = {
              .owner  = THIS_MODULE,
              .name   = "s3c2410-wdt",
+#ifdef CONFIG_OF
+             .of_match_table = s3c2410_wdt_match,
+#endif

The #ifdef can be dropped here now.  .of_match_table is no longer conditional.

The of_match_table is conditional in devicetree/arm and
devicetree/test branches. Are you referring to any other branch?

Thanks,
Thomas.

quoted

      },
 };

--
1.6.6.rc2