]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
board_f: Copy GD to new GD even if relocation disabled
authorMarek Vasut <marek.vasut+renesas@gmail.com>
Sat, 13 Nov 2021 17:34:04 +0000 (18:34 +0100)
committerTom Rini <trini@konsulko.com>
Thu, 18 Nov 2021 23:20:19 +0000 (18:20 -0500)
Even if U-Boot has relocation disabled via GD_FLG_SKIP_RELOC , the
relocated stage of U-Boot still picks GD from new_gd location. The
U-Boot itself is not relocated, but GD might be, so copy the GD to
new GD location even if relocation is disabled.

Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
Cc: Peng Fan <peng.fan@oss.nxp.com>
Cc: Simon Glass <sjg@chromium.org>
Cc: Tom Rini <trini@konsulko.com>
common/board_f.c

index 3dc0eaa59c53eef55dd5be5aa8ba2957176bc3f2..f7ea7c7a1e4507867aaff72510403fed4b6742e3 100644 (file)
@@ -673,30 +673,32 @@ static int reloc_bloblist(void)
 
 static int setup_reloc(void)
 {
-       if (gd->flags & GD_FLG_SKIP_RELOC) {
-               debug("Skipping relocation due to flag\n");
-               return 0;
-       }
-
+       if (!(gd->flags & GD_FLG_SKIP_RELOC)) {
 #ifdef CONFIG_SYS_TEXT_BASE
 #ifdef ARM
-       gd->reloc_off = gd->relocaddr - (unsigned long)__image_copy_start;
+               gd->reloc_off = gd->relocaddr - (unsigned long)__image_copy_start;
 #elif defined(CONFIG_M68K)
-       /*
-        * On all ColdFire arch cpu, monitor code starts always
-        * just after the default vector table location, so at 0x400
-        */
-       gd->reloc_off = gd->relocaddr - (CONFIG_SYS_TEXT_BASE + 0x400);
+               /*
+                * On all ColdFire arch cpu, monitor code starts always
+                * just after the default vector table location, so at 0x400
+                */
+               gd->reloc_off = gd->relocaddr - (CONFIG_SYS_TEXT_BASE + 0x400);
 #elif !defined(CONFIG_SANDBOX)
-       gd->reloc_off = gd->relocaddr - CONFIG_SYS_TEXT_BASE;
+               gd->reloc_off = gd->relocaddr - CONFIG_SYS_TEXT_BASE;
 #endif
 #endif
+       }
+
        memcpy(gd->new_gd, (char *)gd, sizeof(gd_t));
 
-       debug("Relocation Offset is: %08lx\n", gd->reloc_off);
-       debug("Relocating to %08lx, new gd at %08lx, sp at %08lx\n",
-             gd->relocaddr, (ulong)map_to_sysmem(gd->new_gd),
-             gd->start_addr_sp);
+       if (gd->flags & GD_FLG_SKIP_RELOC) {
+               debug("Skipping relocation due to flag\n");
+       } else {
+               debug("Relocation Offset is: %08lx\n", gd->reloc_off);
+               debug("Relocating to %08lx, new gd at %08lx, sp at %08lx\n",
+                     gd->relocaddr, (ulong)map_to_sysmem(gd->new_gd),
+                     gd->start_addr_sp);
+       }
 
        return 0;
 }