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

Simon Glass sjg at chromium.org
Tue Mar 7 00:23:25 CET 2023


Hi Marek,

On Mon, 6 Mar 2023 at 12:07, Marek Vasut <marek.vasut at mailbox.org> wrote:
>
> 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 ?

Yes...I think the helpers were added more recently than this code.

Regards,
Simon


More information about the U-Boot mailing list