Terve, On Thu, 25 Oct 2012, Aaro Koskinen wrote: > On Mon, Oct 15, 2012 at 07:32:33PM -0600, Paul Walmsley wrote: > > The OMAP watchdog timer driver directly calls a function exported by > > code in arch/arm/mach-omap2. This is not good; it tightly couples > > this driver to the mach-omap2 integration code. Instead, add a > > temporary platform_data function pointer to abstract this function > > call. A subsequent patch will convert the watchdog driver to use this > > function pointer. > > Why a function is needed? Reset cause won't change until the next reset, > so it should be enough to read it once during the init and store it into > the platform data. Once the PRM/CM drivers and DT conversion is complete, this driver won't have any platform_data. There won't be any watchdog driver initialization code in arch/arm/mach-omap2. At that point in time, the driver will just call a function from the PRM driver that provides the reset source data. As you say, the watchdog can certainly do this from its probe code; it would be more efficient. Just didn't want to make that change now; seems like it would be best done as a separate optimization patch. - Paul -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
