On Tue, Jan 03, 2012 at 08:01:15PM +0900, Kukjin Kim wrote: > If any problems, please kindly let me know. Right, this looks better. I'm now left with one remaining bit: arch/arm/mach-exynos/include/mach/system.h:#include <plat/system-reset.h> arch/arm/plat-s3c24xx/cpu.c:#include <plat/system-reset.h> arch/arm/mach-s3c64xx/include/mach/system.h:#include <plat/system-reset.h> arch/arm/mach-s3c2410/include/mach/system.h:#include <plat/system-reset.h> and: arch/arm/plat-samsung/include/plat/system-reset.h: ... #include <plat/watchdog-reset.h> static void arch_reset(char mode, const char *cmd) { arch_wdt_reset(); } I assume that with all the patches I now have merged, arch_reset() should never be called on any Samsung platform, and so the include of plat/watchdog-reset.h and call of arch_wdt_reset() can be removed in my "ARM: restart: plat-samsung: remove plat/reset.h and s5p_reset_hook" patch? What about arch/arm/plat-s3c24xx/cpu.c's include of system-reset.h? Does that need to be replaced with watchdog-reset.h? That then leads to "ARM: restart: remove the now empty arch_reset()" removing system-reset.h and all includes of that file? -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html