On Mon, Oct 18, 2021 at 3:02 PM Michal Simek <michal.si...@xilinx.com> wrote:
>
> There is no need to use u-boot.itb as name when this is already recorded in
> CONFIG_SPL_FS_LOAD_PAYLOAD_NAME macro.
>
> Signed-off-by: Michal Simek <michal.si...@xilinx.com>

Reviewed-by: Oleksandr Suvorov <oleksandr.suvo...@foundries.io>


> ---
>
>  board/xilinx/zynqmp/zynqmp.c | 15 +++++++++------
>  1 file changed, 9 insertions(+), 6 deletions(-)
>
> diff --git a/board/xilinx/zynqmp/zynqmp.c b/board/xilinx/zynqmp/zynqmp.c
> index 5fce74d6085a..381f2f82eb8c 100644
> --- a/board/xilinx/zynqmp/zynqmp.c
> +++ b/board/xilinx/zynqmp/zynqmp.c
> @@ -877,20 +877,23 @@ void set_dfu_alt_info(char *interface, char *devstr)
>                 if (!multiboot)
>                         snprintf(buf, DFU_ALT_BUF_LEN,
>                                  "mmc %d:1=boot.bin fat %d 1;"
> -                                "u-boot.itb fat %d 1",
> -                                bootseq, bootseq, bootseq);
> +                                "%s fat %d 1",
> +                                bootseq, bootseq,
> +                                CONFIG_SPL_FS_LOAD_PAYLOAD_NAME, bootseq);
>                 else
>                         snprintf(buf, DFU_ALT_BUF_LEN,
>                                  "mmc %d:1=boot%04d.bin fat %d 1;"
> -                                "u-boot.itb fat %d 1",
> -                                bootseq, multiboot, bootseq, bootseq);
> +                                "%s fat %d 1",
> +                                bootseq, multiboot, bootseq,
> +                                CONFIG_SPL_FS_LOAD_PAYLOAD_NAME, bootseq);
>                 break;
>         case QSPI_MODE_24BIT:
>         case QSPI_MODE_32BIT:
>                 snprintf(buf, DFU_ALT_BUF_LEN,
>                          "sf 0:0=boot.bin raw %x 0x1500000;"
> -                        "u-boot.itb raw 0x%x 0x500000",
> -                        multiboot * SZ_32K, CONFIG_SYS_SPI_U_BOOT_OFFS);
> +                        "%s raw 0x%x 0x500000",
> +                        multiboot * SZ_32K, CONFIG_SPL_FS_LOAD_PAYLOAD_NAME,
> +                        CONFIG_SYS_SPI_U_BOOT_OFFS);
>                 break;
>         default:
>                 return;
> --
> 2.33.1
>


--
Best regards,

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

Reply via email to