[U-Boot] [RFC][PATCH] cmd: fdt: Fix fdt address information after the movement

[Simon Glass]

Hi Marek,

On 24 February 2018 at 19:09, Marek Vasut <marek.vasut at gmail.com> wrote:
> From: Hiroyuki Yokoyama <hiroyuki.yokoyama.vx at renesas.com>
>
> This patch fixes the address information of fdt.
>
> wrong case:
>  => fdt addr 0x48000000
>  => fdt move 0x48000000 0x41000000 0xa000
>  => fdt addr
> The address of the fdt is 48000000
>
> Active address in this case is 0x41000000.
>
> Signed-off-by: Hiroyuki Yokoyama <hiroyuki.yokoyama.vx at renesas.com>
> Signed-off-by: Marek Vasut <marek.vasut+renesas at gmail.com>
> Cc: Hiroyuki Yokoyama <hiroyuki.yokoyama.vx at renesas.com>
> Cc: Nobuhiro Iwamatsu <iwamatsu at nigauri.org>
> Cc: Pantelis Antoniou <pantelis.antoniou at konsulko.com>
> ---
>  cmd/fdt.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/cmd/fdt.c b/cmd/fdt.c
> index b783b0df42..1245bc24eb 100644
> --- a/cmd/fdt.c
> +++ b/cmd/fdt.c
> @@ -204,6 +204,8 @@ static int do_fdt(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
>                         return 1;
>                 }
>                 working_fdt = newaddr;
> +               env_set_hex("fdtaddr", (ulong)working_fdt);

Shouldn't this be map_to_sysmem(working_fdt)?

> +
>  #ifdef CONFIG_OF_SYSTEM_SETUP
>         /* Call the board-specific fixup routine */
>         } else if (strncmp(argv[1], "sys", 3) == 0) {
> --
> 2.16.1

Regards,
Simon