Hi Marek,
On Wed, 1 Mar 2023 at 20:04, Marek Vasut
<marek.vasut+rene...@mailbox.org> wrote:
>
> The 'fdt get addr' is always assumed to be hex value, drop the prefix.
> Since this might break existing users who depend on the existing
> behavior with 0x prefix, this is a separate patch. Revert if this
> breaks anything.
>
> Signed-off-by: Marek Vasut <marek.vasut+rene...@mailbox.org>
> ---
> Cc: Heinrich Schuchardt <heinrich.schucha...@canonical.com>
> Cc: Simon Glass <s...@chromium.org>
> Cc: Tom Rini <tr...@konsulko.com>
> ---
> cmd/fdt.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/cmd/fdt.c b/cmd/fdt.c
> index f38fe909c3e..04b664e652c 100644
> --- a/cmd/fdt.c
> +++ b/cmd/fdt.c
> @@ -478,7 +478,7 @@ static int do_fdt(struct cmd_tbl *cmdtp, int flag, int
> argc, char *const argv[])
> /* Get address */
> char buf[19];
>
> - snprintf(buf, sizeof(buf), "0x%lx",
> + snprintf(buf, sizeof(buf), "%lx",
> (ulong)map_to_sysmem(nodep));
> env_set(var, buf);
> } else if (subcmd[0] == 's') {
> --
> 2.39.2
>
iwc how about using env_sethex() ?
Regards,
Simon
