Hi Kyle,
On Tue, Mar 8, 2011 at 4:37 AM, Kyle Moffett <kyle.d.moff...@boeing.com> wrote:
> Hello everyone,
>
> This patch series creates a generic set of functions available to generic
> code which can be easily hooked per-architecture or per-board. I have
> tried to be very careful that U-Boot builds and runs on all architectures
> for the entire duration of this patchset.
>
> This patchset is designed to produce zero change in behavior *except* for
> one particular scenario: Platforms which perform a "restart" with a simple
> jump to the "_start" entrypoint (or similar) will no longer try to do that
> on panic().
>
> The new functions to be called from generic code are:
>
> int system_restart(void)
> void emergency_restart(void)
>
> The specific platform hooks available are:
>
> int __arch_restart(void)
> int __board_restart(void)
> void __arch_emergency_restart(void)
> void __board_emergency_restart(void)
>
> The first few patches set up the generic functions and hooks with a default
> fallback to the existing "do_reset()" function. Most of the rest are then
> architecture-specific modifications necessary to convert away from the old
> do_reset() prototype. The last patch finally does away with that function.
>
> In my previous discussions with Wolfgang Denk about the __arch_restart()
> hook he requested that it should be mandatory for all architectures to
> implement. Unfortunately the MIPS and MicroBlaze architectures have no
> architectural system-reset code at all, and others have only very limited
> "soft-reboot" (IE: Jump-to-"_start").
>
> Specifically, the MIPS do_reset() function used to look like this:
> _machine_restart();
> printf("*** restart failed ***\n");
>
> When those platforms are fixed then it should be safe to remove the weak
> fallback __arch_restart() function.
>
I recall seeing a patch series similar to this a little while ago. Is this
a Version 2 patch series? If so, what's different?
Regards,
Graeme
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
