David Brownell <[email protected]> writes: > On Tuesday 21 April 2009, Kevin Hilman wrote: >> Russell King - ARM Linux <[email protected]> writes: >> >> > On Wed, Apr 15, 2009 at 10:38:07AM -0500, Kevin Hilman wrote: >> >> @@ -343,6 +353,14 @@ struct sys_timer davinci_timer = { >> >> void davinci_watchdog_reset(void) { >> >> u32 tgcr, wdtcr; >> >> void __iomem *base = IO_ADDRESS(DAVINCI_WDOG_BASE); >> >> + struct device dev; >> >> + struct clk *wd_clk; >> >> + >> >> + dev_set_name(&dev, "watchdog"); >> > >> > This is a (small) memory leak. The name is kmalloc'd now. >> >> OK, I'll rework using the name on the stack. > > Or maybe use the regression fix I sent Saturday: > > http://linux.omap.com/pipermail/davinci-linux-open-source/2009-April/012743.html >
On second thought, I can't use this fix yet since the devices file isn't fully populated. I'll leave the current version, and your patch will be incorporated once I sync up the devices file and the other board files. Kevin _______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
