arc: Introduce a possibility to not relocate U-boot
authorAlexey Brodkin <Alexey.Brodkin@synopsys.com>
Wed, 16 Dec 2015 16:24:10 +0000 (19:24 +0300)
committerAlexey Brodkin <abrodkin@synopsys.com>
Wed, 21 Mar 2018 13:21:34 +0000 (16:21 +0300)
Disabling relocation might be useful on ARC for 2 reasons:
 a) For advanced debugging with Synopsys proprietary MetaWare debugger
    which is capable of accessing much more specific hardware resources
    compared to gdb. For example it may show contents of L1 and L2 caches,
    internal states of some hardware blocks etc.

    But on the downside MetaWare debugger still cannot work with PIE.
    Even though that limitation could be work-arounded with change of ELF's
    header and stripping down all debug info but with it we won't have
    debug info for source-level debugging which is quite inconvenient.

 b) Some platforms which might benefit from usage of U-Boot basically
    don't have enough RAM to accommodate relocation of U-Boot so we
    keep code in flash and use as much of RAM as possible for more
    interesting things.

Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com>
Cc: Simon Glass <sjg@chromium.org>
Cc: Bin Meng <bmeng.cn@gmail.com>
Cc: Heiko Schocher <hs@denx.de>
Cc: York Sun <york.sun@nxp.com>
Cc: Stefan Roese <sr@denx.de>
arch/arc/lib/relocate.c
arch/arc/lib/start.S
common/board_f.c

index 7802f4054594fcfa311225c542eef9256d27c4f0..96b4bd3d8fa6d1a2c83646b0ee7d67a695e821e8 100644 (file)
@@ -17,6 +17,9 @@ int copy_uboot_to_ram(void)
 {
        size_t len = (size_t)&__image_copy_end - (size_t)&__image_copy_start;
 
+       if (gd->flags & GD_FLG_SKIP_RELOC)
+               return 0;
+
        memcpy((void *)gd->relocaddr, (void *)&__image_copy_start, len);
 
        return 0;
@@ -40,6 +43,9 @@ int do_elf_reloc_fixups(void)
        Elf32_Rela *re_src = (Elf32_Rela *)(&__rel_dyn_start);
        Elf32_Rela *re_end = (Elf32_Rela *)(&__rel_dyn_end);
 
+       if (gd->flags & GD_FLG_SKIP_RELOC)
+               return 0;
+
        debug("Section .rela.dyn is located at %08x-%08x\n",
              (unsigned int)re_src, (unsigned int)re_end);
 
index ac9944c4cf3219119fb235d988f7eff90870e357..c78dd001d81d2996cb99ffe6fbc4baae9be2e500 100644 (file)
@@ -78,7 +78,13 @@ ENTRY(_start)
 
        /* Zero the one and only argument of "board_init_f" */
        mov_s   %r0, 0
-       j       board_init_f
+       bl      board_init_f
+
+       /* We only get here if relocation is disabled by GD_FLG_SKIP_RELOC */
+       /* Make sure we don't lose GD overwritten by zero new GD */
+       mov     %r0, %r25
+       mov     %r1, 0
+       bl      board_init_r
 ENDPROC(_start)
 
 /*
index 62588c5bad326f857f128b99f084082352c8528a..ae8bdb7c5c0c577df2d1999603bc5224f8318a97 100644 (file)
@@ -900,7 +900,8 @@ void board_init_f(ulong boot_flags)
                hang();
 
 #if !defined(CONFIG_ARM) && !defined(CONFIG_SANDBOX) && \
-               !defined(CONFIG_EFI_APP) && !CONFIG_IS_ENABLED(X86_64)
+               !defined(CONFIG_EFI_APP) && !CONFIG_IS_ENABLED(X86_64) && \
+               !defined(CONFIG_ARC)
        /* NOTREACHED - jump_to_copy() does not return */
        hang();
 #endif