Hi Chanho, On Mon, 28 Aug 2023 at 21:46, Chanho Park <chanho61.p...@samsung.com> wrote: > > Hi Simon, > > > -----Original Message----- > > From: Simon Glass <s...@chromium.org> > > Sent: Tuesday, August 29, 2023 2:55 AM > > To: Chanho Park <chanho61.p...@samsung.com> > > Cc: Nikhil M Jain <n-ja...@ti.com>; Marek Vasut <ma...@denx.de>; u- > > b...@lists.denx.de > > Subject: Re: [PATCH] spl: bootstage: move bootstage_stash before jumping > > to image > > > > Hi Chanho, > > > > On Mon, 28 Aug 2023 at 03:46, Chanho Park <chanho61.p...@samsung.com> > > wrote: > > > > > > For IH_OS_OPENSBI and IH_OS_LINUX, there is no chance to stash > > > bootstare record because it will not return after jumping to the image. > > > Hence, this patch moves the location of bootstage_stash before jumping > > > to image. > > > > > > > Please fix your implementation instead. You must jump to the OS at the end > > of this function and not before. > > In case of OS_TEE/OPENSBI/LINUX images, they will not be returned to SPL so > they can't get a chance to stash boot stage records. > > > > > In fact I have a patch to move bootstage later in this function! > > Will you post the patch soon? Or it was already posted?
Not posted yet. I'll reply on your new patch. Regards, Simon