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 sjg@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 trini@konsulko.com