[PATCH] xilinx: zynqmp: Save multiboot as variable

[Oleksandr Suvorov]

On Thu, Oct 21, 2021 at 9:59 AM Michal Simek <michal.simek at xilinx.com> wrote:
>
> Save multiboot register as u-boot variable. And use it as primary source
> for composing dfu_alt_info for capsule update. If variable is not defined
>
> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
> ---
>
>  board/xilinx/zynqmp/zynqmp.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/board/xilinx/zynqmp/zynqmp.c b/board/xilinx/zynqmp/zynqmp.c
> index ed6c287df221..5fce74d6085a 100644
> --- a/board/xilinx/zynqmp/zynqmp.c
> +++ b/board/xilinx/zynqmp/zynqmp.c
> @@ -658,6 +658,8 @@ int board_late_init(void)
>         if (ret)
>                 return ret;
>
> +       env_set_hex("multiboot", multi_boot());

multi_boot() can return error -EINVAL. Probably it would be better to
handle this
before setting the env variable?

> +
>         bootmode = zynqmp_get_bootmode();
>
>         puts("Bootmode: ");
> @@ -863,7 +865,7 @@ void set_dfu_alt_info(char *interface, char *devstr)
>
>         memset(buf, 0, sizeof(buf));
>
> -       multiboot = multi_boot();
> +       multiboot = env_get_hex("multiboot", multi_boot());
>         debug("Multiboot: %d\n", multiboot);
>
>         switch (zynqmp_get_bootmode()) {
> --
> 2.33.1
>


-- 
Best regards,

Oleksandr Suvorov
Software Engineer
T: +380 63 8489656
E: oleksandr.suvorov at foundries.io
W: www.foundries.io