[U-Boot] [PATCH] bootm: Add board specific OS preboot hook

Simon Goldschmidt simon.k.r.goldschmidt at gmail.com
Mon Oct 8 05:54:09 UTC 2018


On Thu, Oct 4, 2018 at 9:16 PM Marek Vasut <marex at denx.de> wrote:
>
> Add board-specific hook which is executed before the code hands over
> control to the OS. This lets the board perform some last-minute clean
> ups.
>
> Signed-off-by: Marek Vasut <marex at denx.de>
> Cc: Tom Rini <trini at konsulko.com>

Great idea. There are already some boards (including my private ones)
that misuse arch_preboot_os(). Those could be converted, too.

Reviewed-by: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>

> ---
>  common/bootm_os.c | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/common/bootm_os.c b/common/bootm_os.c
> index f4bd905909..f302135868 100644
> --- a/common/bootm_os.c
> +++ b/common/bootm_os.c
> @@ -505,10 +505,17 @@ __weak void arch_preboot_os(void)
>         /* please define platform specific arch_preboot_os() */
>  }
>
> +/* Allow for board specific config before we boot */
> +__weak void board_preboot_os(void)
> +{
> +       /* please define board specific board_preboot_os() */
> +}
> +
>  int boot_selected_os(int argc, char * const argv[], int state,
>                      bootm_headers_t *images, boot_os_fn *boot_fn)
>  {
>         arch_preboot_os();
> +       board_preboot_os();
>         boot_fn(state, argc, argv, images);
>
>         /* Stand-alone may return when 'autostart' is 'no' */
> --
> 2.18.0
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot


More information about the U-Boot mailing list