On Tue, Mar 31, 2009 at 11:17:57AM -0700, Mark A. Greer wrote: > On Mon, Mar 30, 2009 at 12:53:29PM -0700, Kevin Hilman wrote: > > "Mark A. Greer" <[email protected]> writes: > > > > > From: Mark A. Greer <[email protected]> > > > > > > The watchdog code currently hardcodes the base address > > > of the timer its using. To support new SoCs, make it > > > support timers at any address. Use davinci_soc_info > > > to do this. > > > > > > Signed-off-by: Mark A. Greer <[email protected]> > > > > You should also update the WDT setup from devices.c. This is where > > it sets up the actual watchdog driver. > > > > Now that it's in soc_info, it could be done in common.c instead of > > devices.c. > > You mean doing the platform_device_register(), right? If so, I think > I'll put it in the soc files then because the da830 stuff does the > registering a bit differently (see devices-da830.c:da830_add_devices()).
Actually, that doesn't make a whole lot of sense either. I'd like to just leave it as it is. How vigorously to you object to that? Mark -- _______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
