Hi, 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. A. -- 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