[PATCH] cmd: fdt: Drop the 0x prefix

[Marek Vasut]

On 3/6/23 18:53, Simon Glass wrote:
> Hi Marek,
> 
> On Wed, 1 Mar 2023 at 20:04, Marek Vasut
> <marek.vasut+renesas at 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+renesas at mailbox.org>
>> ---
>> Cc: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
>> Cc: Simon Glass <sjg at chromium.org>
>> Cc: Tom Rini <trini at 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() ?

The 'env get size' 's' case below could likely use similar treatment , 
do I read it right ?