[U-Boot] [PATCH 2/3] mpc85xx: Add a board-specific restart hook

[Mike Frysinger]

On Wednesday 19 October 2011 14:26:03 Moffett, Kyle D wrote:
> On Oct 18, 2011, at 23:20, Mike Frysinger wrote:
> > On Tuesday 18 October 2011 19:41:23 Kyle Moffett wrote:
> >> +int do_reset(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
> >> +{
> >> 
> >> 	unsigned long val, msr;
> >> 
> >> +	/* Allow boards to override the reset */
> >> +	int err = __board_restart();
> >> +	if (err)
> >> +		return err;
> > 
> > i thought we decided that do_reset() shouldn't return
> 
> For our hardware we have to coordinate reset between both CPUs on the
> same physical board, so a "reset" command may hang indefinitely waiting
> for the other CPU (IE: If it refuses to shutdown in Linux or is running
> U-Boot).
> 
> So for user convenience I need to be able to Ctrl-C the communication.
> Since "reset" is basically just like any other U-Boot shell command,
> (except with some side-effects) it seems reasonable to allow a board
> handler to return an error instead of resetting.

i thought this came up before and we said "no".  but Wolfgang was more 
involved in that discussion, so he should be able to better say.

this is probably in the mail archives somewhere too ...
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20111019/aecb00e2/attachment.pgp