5 Jul
2013
5 Jul
'13
9:52 p.m.
Hi Tom,
On Fri, Jul 5, 2013 at 5:59 AM, Tom Rini trini@ti.com wrote:
On Thu, Jul 04, 2013 at 01:17:07PM -0700, Simon Glass wrote:
A recent bootm fix left the error path incomplete. Reinstate this so that failures in bootm stages are handled properly.
Signed-off-by: Simon Glass sjg@chromium.org
Changes in v2:
- Correct checking in the no-error case
OK, this conflicts with the change I posted (and pushed later than I thought I had). Can you confirm the code is good in mainline now? Thanks!
It's close, but I think it still needs this near the end of do_bootm_states(), something like:
else if (ret == BOOTM_ERR_RESET) do_reset(cmdtp, flag, argc, argv); + else if (ret) + puts("subcommand not supported\n"); return ret;
If you agree, I can prepare a patch as part of the bootz update.
Regards, Simon