Hi Chanho, On Mon, 28 Aug 2023 at 21:46, Chanho Park <[email protected]> wrote: > > Hi Simon, > > > -----Original Message----- > > From: Simon Glass <[email protected]> > > Sent: Tuesday, August 29, 2023 2:55 AM > > To: Chanho Park <[email protected]> > > Cc: Nikhil M Jain <[email protected]>; Marek Vasut <[email protected]>; u- > > [email protected] > > Subject: Re: [PATCH] spl: bootstage: move bootstage_stash before jumping > > to image > > > > Hi Chanho, > > > > On Mon, 28 Aug 2023 at 03:46, Chanho Park <[email protected]> > > 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

