Re: [PATCH] cmd: fdt: Drop the 0x prefix
Hi Marek, On Mon, 6 Mar 2023 at 12:07, Marek Vasut wrote: > > On 3/6/23 18:53, Simon Glass wrote: > > Hi Marek, > > > > On Wed, 1 Mar 2023 at 20:04, Marek Vasut > > 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 > >> --- > >> Cc: Heinrich Schuchardt > >> Cc: Simon Glass > >> Cc: Tom Rini > >> --- > >> cmd/fdt.c | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> Applied to u-boot-dm/next, thanks!
Re: [PATCH] cmd: fdt: Drop the 0x prefix
Hi Marek,
On Mon, 6 Mar 2023 at 12:07, Marek Vasut wrote:
>
> On 3/6/23 18:53, Simon Glass wrote:
> > Hi Marek,
> >
> > On Wed, 1 Mar 2023 at 20:04, Marek Vasut
> > 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
> >> ---
> >> Cc: Heinrich Schuchardt
> >> Cc: Simon Glass
> >> Cc: Tom Rini
> >> ---
> >> 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
Re: [PATCH] cmd: fdt: Drop the 0x prefix
On 3/6/23 18:53, Simon Glass wrote:
Hi Marek,
On Wed, 1 Mar 2023 at 20:04, Marek Vasut
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
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
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 ?
Re: [PATCH] cmd: fdt: Drop the 0x prefix
Hi Marek,
On Wed, 1 Mar 2023 at 20:04, Marek Vasut
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
> ---
> Cc: Heinrich Schuchardt
> Cc: Simon Glass
> Cc: Tom Rini
> ---
> 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
[PATCH] cmd: fdt: Drop the 0x prefix
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
---
Cc: Heinrich Schuchardt
Cc: Simon Glass
Cc: Tom Rini
---
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
