Re: [PATCH] cmd: add FDT setup for bootelf by flag

On Mon, Feb 12, 2024 at 09:41:17PM +0300, Maxim Moskalets wrote:

Added the ability to use FDT for ELF applications, required to run some OS. To make FDT setup, you need to set the elf_needed_fdt environment variable to a value like y or yes.

Signed-off-by: Maxim Moskalets Maxim.Moskalets@kaspersky.com

Cc: Tom Rini trini@konsulko.com

cmd/elf.c | 14 ++++++++++++++ env/common.c | 5 +++++ include/env.h | 7 +++++++ 3 files changed, 26 insertions(+)

Is there some change compared with the last version of this patch? And, I was thinking of how to reply to the previous one and I think my question is why don't we just use a flag being passed to bootelf instead of this? Doing bootelf addr -d fdt_addr seems clearer than bootelf addr1 addr2 and checking the environment.