[PATCH] xilinx: versal: Add new versal loadpdi command

[Michal Simek]

Hi,

st 24. 6. 2020 v 13:00 odesílatel Michal Simek <michal.simek at xilinx.com> napsal:
>
> From: T Karthik Reddy <t.karthik.reddy at xilinx.com>
>
> Versal loadpdi command is used for loading secure & non-secure
> pdi images.
>
> Signed-off-by: T Karthik Reddy <t.karthik.reddy at xilinx.com>
> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
> ---
>
>  board/xilinx/versal/Makefile |   1 +
>  board/xilinx/versal/cmds.c   | 102 +++++++++++++++++++++++++++++++++++
>  2 files changed, 103 insertions(+)
>  create mode 100644 board/xilinx/versal/cmds.c
>
> diff --git a/board/xilinx/versal/Makefile b/board/xilinx/versal/Makefile
> index e9307d7fa690..1fee0ef01b77 100644
> --- a/board/xilinx/versal/Makefile
> +++ b/board/xilinx/versal/Makefile
> @@ -5,4 +5,5 @@
>  #
>
>  obj-y  := board.o
> +obj-y  += cmds.o
>  obj-y  += ../common/board.o
> diff --git a/board/xilinx/versal/cmds.c b/board/xilinx/versal/cmds.c
> new file mode 100644
> index 000000000000..13720e99a26d
> --- /dev/null
> +++ b/board/xilinx/versal/cmds.c
> @@ -0,0 +1,102 @@
> +// SPDX-License-Identifier: GPL-2.0
> +/*
> + * (C) Copyright 2020 Xilinx, Inc.
> + * Michal Simek <michal.simek at xilinx.com>
> + */
> +
> +#include <cpu_func.h>
> +#include <common.h>
> +#include <versalpl.h>
> +#include <zynqmp_firmware.h>
> +
> +static int do_versal_load_pdi(cmd_tbl_t *cmdtp, int flag, int argc,
> +                             char * const argv[])
> +{
> +       u32 buf_lo, buf_hi;
> +       u32 ret_payload[5];
> +       ulong addr, *pdi_buf;
> +       size_t len;
> +       int ret;
> +
> +       if (argc != cmdtp->maxargs) {
> +               debug("pdi_load: incorrect parameters passed\n");
> +               return CMD_RET_USAGE;
> +       }
> +
> +       addr = simple_strtol(argv[2], NULL, 16);
> +       if (!addr) {
> +               debug("pdi_load: zero pdi_data address\n");
> +               return CMD_RET_USAGE;
> +       }
> +
> +       len = simple_strtoul(argv[3], NULL, 16);
> +       if (!len) {
> +               debug("pdi_load: zero size\n");
> +               return CMD_RET_USAGE;
> +       }
> +
> +       pdi_buf = (ulong *)ALIGN((ulong)addr, ARCH_DMA_MINALIGN);
> +       if ((ulong)addr != (ulong)pdi_buf) {
> +               memcpy((void *)pdi_buf, (void *)addr, len);
> +               debug("Pdi addr:0x%lx aligned to 0x%lx\n",
> +                     addr, (ulong)pdi_buf);
> +       }
> +
> +       flush_dcache_range((ulong)pdi_buf, (ulong)pdi_buf + len);
> +
> +       buf_lo = lower_32_bits((ulong)pdi_buf);
> +       buf_hi = upper_32_bits((ulong)pdi_buf);
> +
> +       ret = xilinx_pm_request(VERSAL_PM_LOAD_PDI, VERSAL_PM_PDI_TYPE, buf_lo,
> +                               buf_hi, 0, ret_payload);
> +       if (ret)
> +               printf("PDI load failed with err: 0x%08x\n", ret);
> +
> +       return ret;
> +}
> +
> +static cmd_tbl_t cmd_versal_sub[] = {
> +       U_BOOT_CMD_MKENT(loadpdi, 4, 1, do_versal_load_pdi, "", ""),
> +};
> +
> +/**
> + * do_versal - Handle the "versal" command-line command
> + * @cmdtp:      Command data struct pointer
> + * @flag:       Command flag
> + * @argc:       Command-line argument count
> + * @argv:       Array of command-line arguments
> + *
> + * Processes the versal specific commands
> + *
> + * Return: return 0 on success, Error value if command fails.
> + * CMD_RET_USAGE incase of incorrect/missing parameters.
> + */
> +static int do_versal(cmd_tbl_t *cmdtp, int flag, int argc,
> +                    char *const argv[])
> +{
> +       cmd_tbl_t *c;
> +       int ret = CMD_RET_USAGE;
> +
> +       if (argc < 2)
> +               return CMD_RET_USAGE;
> +
> +       c = find_cmd_tbl(argv[1], &cmd_versal_sub[0],
> +                        ARRAY_SIZE(cmd_versal_sub));
> +       if (c)
> +               ret = c->cmd(c, flag, argc, argv);
> +
> +       return cmd_process_error(c, ret);
> +}
> +
> +#ifdef CONFIG_SYS_LONGHELP
> +static char versal_help_text[] =
> +       "loadpdi addr len - Load pdi image\n"
> +       "load pdi image at ddr address 'addr' with pdi image size 'len'\n"
> +;
> +#endif
> +
> +U_BOOT_CMD(versal, 4, 1, do_versal,
> +          "versal sub-system",
> +          versal_help_text
> +)
> +
> --
> 2.27.0
>

Please ignore this patch. CI loop found some issues with it and v2 is required.

Thanks,
Michal


-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs