[U-Boot] [RFC PATCH 04/20] sandbox: Add cpu files

Mike Frysinger vapier at gentoo.org
Mon Sep 26 06:48:40 CEST 2011


On Sunday, September 25, 2011 16:18:32 Simon Glass wrote:
> On Sun, Sep 25, 2011 at 12:25 PM, Wolfgang Denk wrote:
> > Simon Glass wrote:
> >> > do_reset() is not supposed to return
> >> 
> >> I have adjusted the function meaning (which luckily for me was not
> >> defined) so that it can return -1 on failure. This makes my code
> >> correct :-)
> >> 
> >> I think it is reasonable to provide a reset function which might not
> >> be able to do its job. That is the current state of sandbox.
> > 
> > No, I don't want to change the current definition of reset().
> 
> OK.
> 
> > And "not able to do the job" is something different than
> > "unimplemented".
> > 
> > Why cannot we do a real reset here? Re-exec'in the running binary or
> > performing a longjmp() to the start might be ideas how to implement
> > this.
> 
> While this could be done I believe that it might be possible /
> desirable to exit out of the main loop, rather than longjmp or
> re-exec. I have not implemented it because I have not got to that bit
> yet and don't want to put time into a solution I will throw away.
> 
> I'm happy to just put:
> 
> while (1) ;
> 
> in the reset code if you like?

i would expect "reset" in the sandbox to "exit(1)".  how else would you exit ?
-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/20110926/018b4d6e/attachment.pgp 


More information about the U-Boot mailing list