On Friday 19 September 2008, Felipe Balbi wrote: > +static struct platform_device *omap_wdt_dev; > + > ... > +struct omap_wdt_dev { > + void __iomem *base; /* physical */ > + struct device *dev; You don't need both omap_wdt_dev (platform device) and omap_wdt_dev.dev (hmm, never used). In fact the former isn't needed either ... its role seems to be ensure only one watchdog device gets bound, which is more naturally done by not registering more than one such platform device. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html