On Thu, Dec 14, 2023 at 09:50:17AM -0700, Simon Glass wrote: > Some OS functions require the arguments to the 'bootm' command. This is > inconvenient for two reasons. > > Firstly, there may not be any actual command, if CMDLINE is not enabled > and programmatic boot is being used. > > Secondly, most functions don't require the arguments, so it is > inefficient to pass them when not needed. For example it increases code > size. > > Create a new struct which holds the arguments, which can be used if > needed. > > Add the images pointer as well, since this is commonly needed. > > Signed-off-by: Simon Glass <s...@chromium.org> [snip] > @@ -39,7 +53,7 @@ typedef int boot_os_fn(int flag, int argc, char *const > argv[], > extern boot_os_fn do_bootm_linux; > extern boot_os_fn do_bootm_vxworks; > > -int do_bootelf(struct cmd_tbl *cmdtp, int flag, int argc, char *const > argv[]); > +int do_bootelf(struct cmd_tbl *cmdtp, int fglag, int argc, char *const > argv[]);
Typo, can just drop when applying if there's no v3. Reviewed-by: Tom Rini <tr...@konsulko.com> -- Tom
signature.asc
Description: PGP signature
