[PATCH 2/2] cmd: booti: adjust the print format

Simon Glass sjg at chromium.org
Thu Aug 29 16:05:00 CEST 2024


Hi Dario,

On Sun, 25 Aug 2024 at 06:26, Dario Binacchi
<dario.binacchi at amarulasolutions.com> wrote:
>
> All three addresses printed are in hexadecimal format, but only the
> first two have the "0x" prefix. The patch aligns the format of the
> "end" address with the other two by adding the "0x" prefix.
>
> Signed-off-by: Dario Binacchi <dario.binacchi at amarulasolutions.com>
>
> ---
>
>  cmd/booti.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/cmd/booti.c b/cmd/booti.c
> index 62b19e834366..ea811244a0a9 100644
> --- a/cmd/booti.c
> +++ b/cmd/booti.c
> @@ -78,7 +78,7 @@ static int booti_start(struct bootm_info *bmi)
>
>         /* Handle BOOTM_STATE_LOADOS */
>         if (relocated_addr != ld) {
> -               printf("Moving Image from 0x%lx to 0x%lx, end=%lx\n", ld,
> +               printf("Moving Image from 0x%lx to 0x%lx, end=0x%lx\n", ld,
>                        relocated_addr, relocated_addr + image_size);
>                 memmove((void *)relocated_addr, (void *)ld, image_size);
>         }
> --
> 2.43.0
>

I really don't like this...numbers are hex in U-Boot and this just
adds confusion.

Regards,
Simon


More information about the U-Boot mailing list