Hi Bin, On 2 December 2015 at 22:11, Bin Meng <bmeng...@gmail.com> wrote: > 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.
Eek. Nasty unimplemented specs! Acked-by: Simon Glass <s...@chromium.org> > >>> >>> 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
