Hi guys, On Sun, 22 Dec 2024 at 06:17, Ilias Apalodimas <[email protected]> wrote: > > Hi Heinrich > > On Sun, 22 Dec 2024 at 14:45, Heinrich Schuchardt <[email protected]> wrote: > > > > On 12/19/24 03:38, Simon Glass wrote: > > > This uses a few global variables at present. With the bootflow we have > > > the required parameters, so add a function which accepts these. Update > > > the existing function to call the new one with the globals. > > > > > > - Rename efi_binary_run_() to _efi_binary_run() > > > > > > Signed-off-by: Simon Glass <[email protected]> > > > --- > > > > > > (no changes since v1) > > > > > > lib/efi_loader/efi_bootbin.c | 29 +++++++++++++++++++++++++---- > > > 1 file changed, 25 insertions(+), 4 deletions(-) > > > > > > diff --git a/lib/efi_loader/efi_bootbin.c b/lib/efi_loader/efi_bootbin.c > > > index b42ff3cb4e0..f7aedeaab8a 100644 > > > --- a/lib/efi_loader/efi_bootbin.c > > > +++ b/lib/efi_loader/efi_bootbin.c > > > @@ -228,18 +228,22 @@ out: > > > } > > > > > > /** > > > - * efi_binary_run() - run loaded UEFI image > > > + * efi_binary_run_() - run loaded UEFI image > > > * > > > * @image_ptr: memory address of the UEFI image > > > * @size: size of the UEFI image > > > * @fdt: device-tree > > > + * @device: EFI device-path > > > + * @image: EFI image-path > > > * > > > * Execute an EFI binary image loaded at @image. > > > * @size may be zero if the binary is loaded with U-Boot load command. > > > * > > > * Return: status code > > > */ > > > -efi_status_t efi_binary_run(void *image_ptr, size_t size, void *fdt) > > > +efi_status_t _efi_binary_run(void *image_ptr, size_t size, void *fdt, > > > + struct efi_device_path *device, > > > + struct efi_device_path *image) > > > > A non-static function starting with underscore? Please, avoid this. > > I was the one who requested this, the v1 had it as efi_binary_run_, > but i a dont mind any of those
Well I'm not changing it again :-) Heinrich you are free to send a patch later. Regards, Simon
