[PATCH v4] cmd: add FDT setup for bootelf by flag
Maxim M. Moskalets
maximmosk4 at gmail.com
Thu Feb 29 20:44:59 CET 2024
Hello Tom,
On 26.02.2024 21:35, Maxim Moskalets wrote:
> From: Maxim Moskalets <maximmosk4 at gmail.com>
>
> Added the ability to use FDT for ELF applications, required to run some
> OS. To make FDT setup, you need to set the -d fdt_addr_r cmd option for
> bootelf command.
>
> Signed-off-by: Maxim Moskalets <Maxim.Moskalets at kaspersky.com>
>
> Cc: Tom Rini <trini at konsulko.com>
> ---
> cmd/elf.c | 27 ++++++++++++++++++++++++---
> 1 file changed, 24 insertions(+), 3 deletions(-)
>
> diff --git a/cmd/elf.c b/cmd/elf.c
> index b7b9f506a5..2fa28448c0 100644
> --- a/cmd/elf.c
> +++ b/cmd/elf.c
> @@ -38,6 +38,8 @@ static unsigned long do_bootelf_exec(ulong (*entry)(int, char * const[]),
> /* Interpreter command to boot an arbitrary ELF image from memory */
> int do_bootelf(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
> {
> + struct bootm_headers img = {0};
> + unsigned long fdt_addr = 0; /* Address of the FDT */
> unsigned long addr; /* Address of the ELF image */
> unsigned long rc; /* Return value from user code */
> char *sload = NULL;
> @@ -46,13 +48,23 @@ int do_bootelf(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
> /* Consume 'bootelf' */
> argc--; argv++;
>
> - /* Check for flag. */
> + /* Check for [-p|-s] flag. */
> if (argc >= 1 && (argv[0][0] == '-' && \
> (argv[0][1] == 'p' || argv[0][1] == 's'))) {
> sload = argv[0];
> /* Consume flag. */
> argc--; argv++;
> }
> +
> + /* Check for [-d fdt_addr_r] option. */
> + if ((argc >= 2) && (argv[0][0] == '-') && (argv[0][1] == 'd')) {
> + if (strict_strtoul(argv[1], 16, &fdt_addr) != 0)
> + return CMD_RET_USAGE;
> + /* Consume option. */
> + argc -= 2;
> + argv += 2;
> + }
> +
> /* Check for address. */
> if (argc >= 1 && strict_strtoul(argv[0], 16, &addr) != -EINVAL) {
> /* Consume address */
> @@ -68,6 +80,14 @@ int do_bootelf(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
> else
> addr = load_elf_image_shdr(addr);
>
> + if (fdt_addr) {
> + printf("## Setting up FDT at 0x%08lx ...\n", fdt_addr);
> + flush();
> +
> + if (image_setup_libfdt(&img, (void *)fdt_addr, NULL))
> + return 1;
> + }
> +
> if (!env_get_autostart())
> return rcode;
>
> @@ -298,9 +318,10 @@ int do_bootvx(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
> U_BOOT_CMD(
> bootelf, CONFIG_SYS_MAXARGS, 0, do_bootelf,
> "Boot from an ELF image in memory",
> - "[-p|-s] [address]\n"
> + "[-p|-s] [-d fdt_addr_r] [address]\n"
> "\t- load ELF image at [address] via program headers (-p)\n"
> - "\t or via section headers (-s)"
> + "\t or via section headers (-s)\n"
> + "\t- setup FDT image at [fdt_addr_r] (-d)"
> );
>
> U_BOOT_CMD(
Is it ok to merge?
More information about the U-Boot
mailing list