On 13 September 2017 at 09:58, Tom Rini <tr...@konsulko.com> wrote: > On Wed, Sep 13, 2017 at 06:24:24PM +0800, Kever Yang wrote: >> Since we may jump to next stage like ATF/OP-TEE instead of U-Boot, >> we need to stash the bootstage info before it. >> >> Signed-off-by: Kever Yang <kever.y...@rock-chips.com> >> --- >> >> common/spl/spl.c | 18 +++++++++--------- >> 1 file changed, 9 insertions(+), 9 deletions(-) >> >> diff --git a/common/spl/spl.c b/common/spl/spl.c >> index 85e2e88..d156e74 100644 >> --- a/common/spl/spl.c >> +++ b/common/spl/spl.c >> @@ -424,6 +424,15 @@ void board_init_r(gd_t *dummy1, ulong dummy2) >> debug("SPL malloc() used %#lx bytes (%ld KB)\n", gd->malloc_ptr, >> gd->malloc_ptr / 1024); >> #endif >> +#ifdef CONFIG_BOOTSTAGE_STASH >> + int ret; >> + >> + bootstage_mark_name(BOOTSTAGE_ID_END_SPL, "end_spl"); >> + ret = bootstage_stash((void *)CONFIG_BOOTSTAGE_STASH_ADDR, >> + CONFIG_BOOTSTAGE_STASH_SIZE); >> + if (ret) >> + debug("Failed to stash bootstage: err=%d\n", ret); >> +#endif >> >> if (CONFIG_IS_ENABLED(ATF_SUPPORT)) { >> debug("loaded - jumping to U-Boot via ATF BL31.\n"); >> @@ -436,15 +445,6 @@ void board_init_r(gd_t *dummy1, ulong dummy2) >> } >> >> debug("loaded - jumping to U-Boot...\n"); >> -#ifdef CONFIG_BOOTSTAGE_STASH >> - int ret; >> - >> - bootstage_mark_name(BOOTSTAGE_ID_END_SPL, "end_spl"); >> - ret = bootstage_stash((void *)CONFIG_BOOTSTAGE_STASH_ADDR, >> - CONFIG_BOOTSTAGE_STASH_SIZE); >> - if (ret) >> - debug("Failed to stash bootstage: err=%d\n", ret); >> -#endif >> spl_board_prepare_for_boot(); >> jump_to_image_no_args(&spl_image); >> } > > Simon, does this seem right to you? Thanks!
Yes. I think the commit message would be better if it mentioned that the code is moved, not added. Really, it is the CONFIG_IS_ENABLED(ATF_SUPPORT) that is being moved I think. Reviewed-by: Simon Glass <s...@chromium.org> Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot