On Tue, Jun 19, 2012 at 12:29:28PM +0200, Juergen Beisert wrote: > Hi list, > > currently we use the watchdog to make the "reset" command work on the > i.XM23/i.MX28 CPUs. This collides if someone wants to use the watchdog in a > regular manner, as Barebox cannot distinguish if the reset was a result of > the "reset" command (or "reboot" from Linux) or if something strange happens > and the watchdog has barked to reboot and try again (maybe in a different > manner). > > The i.XM23/i.MX28 CPUs are coming with a bit in a special register in > the "clock generation and control" unit to hard reset the whole SoC. This > could be used instead to make the "reset" command work. But the register > offsets are different in both SoCs. So: > > - using ifdefs to distinguish both SoCs in "arch/arm/mach-mxs/reset-imx.c" > - or moving the reset_cpu() function into "arch/arm/mach-mxs/speed-imx*.c" > where all the required symbols and correct offset are already defined on a > per SoC base? Or add a imx23.c and imx28.c where we could collect the SoC specific code. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
