On Tue, Jan 19, 2010 at 16:42, Joe Perches wrote: > On Tue, 2010-01-19 at 22:17 +0100, Wim Van Sebroeck wrote: >> -static struct watchdog_info at32_wdt_info = { >> +static const struct watchdog_info at32_wdt_info = { > > It'd be good to use a consistent structure name: > > static const struct watchdog_info ident = { > etc... > } i'd agree (obviously i'm partial to the "xxx_wdt_info" form), but i'd worry about this after the watchdog implementations are finally unified ... might be less code needing standardization at that point -mike -- 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