Hi Simon, On Thu, Dec 3, 2015 at 5:05 AM, Simon Glass <s...@chromium.org> wrote: > Hi Bin, > > On 2 December 2015 at 01:58, Bin Meng <bmeng...@gmail.com> wrote: >> There is no need to pass shared_data to fsp_continue() so we can >> remove unnecessary codes that simplifies the function a lot. > > Is this because shared_data is effectively useful for a bootloader due > to the stack being changed? >
The removed codes were originally from Queensbay FSP package. Looks Intel's intention was to help stack migration, but I doubt they ever worked. We don't support stack migration after fsp_init() hence these codes are never needed. >> >> Signed-off-by: Bin Meng <bmeng...@gmail.com> >> --- >> >> arch/x86/include/asm/fsp/fsp_support.h | 4 +--- >> arch/x86/lib/fsp/fsp_support.c | 29 ++++++----------------------- >> 2 files changed, 7 insertions(+), 26 deletions(-) >> > > [snip] > > Regards, > Simon Regards, Bin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
