[PATCH v2 16/21] bootm: Tidy up boot_selected_os()

[Simon Glass]

Use struct bootm_info with this function, to avoiding needing to
create a new one.

Signed-off-by: Simon Glass <s...@chromium.org>
---

(no changes since v1)

 boot/bootm.c    |  6 ++----
 boot/bootm_os.c | 11 +++--------
 include/bootm.h |  3 +--
 3 files changed, 6 insertions(+), 14 deletions(-)

diff --git a/boot/bootm.c b/boot/bootm.c
index bd208c0493ca..f9f0c9782e42 100644
--- a/boot/bootm.c
+++ b/boot/bootm.c
@@ -1092,8 +1092,7 @@ int bootm_run_states(struct bootm_info *bmi, int states)
        if (!ret && (states & BOOTM_STATE_OS_FAKE_GO)) {
                char *cmd_list = env_get("fakegocmd");
 
-               ret = boot_selected_os(bmi->argc, bmi->argv,
-                                      BOOTM_STATE_OS_FAKE_GO, images, boot_fn);
+               ret = boot_selected_os(BOOTM_STATE_OS_FAKE_GO, bmi, boot_fn);
                if (!ret && cmd_list)
                        ret = run_command_list(cmd_list, -1, 0);
        }
@@ -1107,8 +1106,7 @@ int bootm_run_states(struct bootm_info *bmi, int states)
 
        /* Now run the OS! We hope this doesn't return */
        if (!ret && (states & BOOTM_STATE_OS_GO))
-               ret = boot_selected_os(bmi->argc, bmi->argv, BOOTM_STATE_OS_GO,
-                                      images, boot_fn);
+               ret = boot_selected_os(BOOTM_STATE_OS_GO, bmi, boot_fn);
 
        /* Deal with any fallout */
 err:
diff --git a/boot/bootm_os.c b/boot/bootm_os.c
index 4f547b1b1148..47a5fd78fb6a 100644
--- a/boot/bootm_os.c
+++ b/boot/bootm_os.c
@@ -566,20 +566,15 @@ __weak void board_preboot_os(void)
        /* please define board specific board_preboot_os() */
 }
 
-int boot_selected_os(int argc, char *const argv[], int state,
-                    struct bootm_headers *images, boot_os_fn *boot_fn)
+int boot_selected_os(int state, struct bootm_info *bmi, boot_os_fn *boot_fn)
 {
-       struct bootm_info bmi;
        arch_preboot_os();
        board_preboot_os();
 
-       bmi.argc = argc;
-       bmi.argv = argv;
-       bmi.images = images;
-       boot_fn(state, &bmi);
+       boot_fn(state, bmi);
 
        /* Stand-alone may return when 'autostart' is 'no' */
-       if (images->os.type == IH_TYPE_STANDALONE ||
+       if (bmi->images->os.type == IH_TYPE_STANDALONE ||
            IS_ENABLED(CONFIG_SANDBOX) ||
            state == BOOTM_STATE_OS_FAKE_GO) /* We expect to return */
                return 0;
diff --git a/include/bootm.h b/include/bootm.h
index 45109f96695d..d0749dc5257a 100644
--- a/include/bootm.h
+++ b/include/bootm.h
@@ -91,8 +91,7 @@ boot_os_fn *bootm_os_get_boot_func(int os);
 int bootm_host_load_images(const void *fit, int cfg_noffset);
 #endif
 
-int boot_selected_os(int argc, char *const argv[], int state,
-                    struct bootm_headers *images, boot_os_fn *boot_fn);
+int boot_selected_os(int state, struct bootm_info *bmi, boot_os_fn *boot_fn);
 
 ulong bootm_disable_interrupts(void);
 
-- 
2.43.0.472.g3155946c3a-goog